chore(ci): align black target to py310 and apply formatting

Author: udaycodespace

.github/workflows/ci.yml

@@ -88,5 +88,5 @@ jobs:
 
     - name: Check code formatting
       run: |
-        black --check app/ core/
+        black --check --target-version py310 app/ core/
       continue-on-error: true

app/app.py

@@ -43,32 +43,33 @@
 zkp_manager = ZKPManager(crypto_manager)
 mailer = None  # Initialized inside create_app
 
+
 def init_extensions(app):
     """Initialize third-party extensions and global state"""
     setup_logging()
     load_dotenv()
-    
+
     app.config.from_object(Config)
     app.config["SQLALCHEMY_DATABASE_URI"] = os.environ.get("DATABASE_URL", "sqlite:///credentials.db")
     app.config["SQLALCHEMY_ENGINE_OPTIONS"] = {"pool_recycle": 300, "pool_pre_ping": True}
-    
+
     init_database(app)
-    
+
     global mailer
     mailer = CredifyMailer(app)
-    
+
     blockchain.difficulty = app.config.get("BLOCKCHAIN_DIFFICULTY", 0)
     blockchain.VALIDATORS = app.config.get("VALIDATOR_USERNAMES", ["admin", "issuer1"])
-    
+
     with app.app_context():
         blockchain.load_blockchain()
         if not blockchain.chain:
             blockchain.create_genesis_block()
-        
+
         # P2P multi-node init
-        peer_nodes_env = os.environ.get('PEER_NODES', '')
+        peer_nodes_env = os.environ.get("PEER_NODES", "")
         if peer_nodes_env:
-            for peer in peer_nodes_env.split(','):
+            for peer in peer_nodes_env.split(","):
                 if peer.strip():
                     try:
                         blockchain.register_node(peer.strip())
@@ -77,6 +78,7 @@ def init_extensions(app):
             if blockchain.nodes:
                 threading.Thread(target=_initial_sync, args=(app,), daemon=True).start()
 
+
 def _initial_sync(app):
     """Background peer synchronization task"""
     time.sleep(5)
@@ -88,6 +90,7 @@ def _initial_sync(app):
         except Exception as e:
             logging.error(f"Sync error: {e}")
 
+
 def register_blueprints(app):
     """Register all modular routing blueprints"""
     from app.blueprints.auth.routes import auth_bp
@@ -96,25 +99,27 @@ def register_blueprints(app):
     from app.blueprints.verifier.routes import verifier_bp
     from app.blueprints.admin.routes import admin_bp
     from app.blueprints.api.routes import api_bp
-    
+
     app.register_blueprint(auth_bp)
     app.register_blueprint(issuer_bp)
     app.register_blueprint(holder_bp)
     app.register_blueprint(verifier_bp)
     app.register_blueprint(admin_bp)
     app.register_blueprint(api_bp)
 
+
 def create_app():
     """Application Factory Pattern"""
-    app = Flask(__name__, template_folder='../templates', static_folder='../static')
+    app = Flask(__name__, template_folder="../templates", static_folder="../static")
     app.secret_key = os.environ.get("SESSION_SECRET", "dev-secret-key-change-in-production")
     cors_origins = os.environ.get("CORS_ORIGINS", "*")
     CORS(app, resources={r"/api/*": {"origins": cors_origins}}, supports_credentials=False)
-    
+
     init_extensions(app)
     register_blueprints(app)
     return app
 
+
 if __name__ == "__main__":
     app_instance = create_app()
-    app_instance.run(host='0.0.0.0', port=5000, debug=True)
+    app_instance.run(host="0.0.0.0", port=5000, debug=True)

app/auth.py

@@ -25,37 +25,44 @@
 
 def login_required(f):
     """Decorator to require login for a route"""
+
     @wraps(f)
     def decorated_function(*args, **kwargs):
-        if 'user_id' not in session:
-            flash('Please login to access this page', 'warning')
-            return redirect(url_for('auth.login'))
+        if "user_id" not in session:
+            flash("Please login to access this page", "warning")
+            return redirect(url_for("auth.login"))
         return f(*args, **kwargs)
+
     return decorated_function
 
 
 def role_required(role):
     """Decorator to require specific role for a route"""
+
     def decorator(f):
         @wraps(f)
         def decorated_function(*args, **kwargs):
             # Check if user is logged in
-            if 'user_id' not in session:
-                flash(f'Please login as {role.title()} to access this portal.', 'warning')
-                if role == 'issuer':
-                    return redirect(url_for('issuer.issuer'))
-                elif role == 'student':
-                    return redirect(url_for('holder.holder'))
-                return redirect(url_for('auth.login'))
-            
+            if "user_id" not in session:
+                flash(f"Please login as {role.title()} to access this portal.", "warning")
+                if role == "issuer":
+                    return redirect(url_for("issuer.issuer"))
+                elif role == "student":
+                    return redirect(url_for("holder.holder"))
+                return redirect(url_for("auth.login"))
+
             # Check user role
-            user_role = session.get('role')
+            user_role = session.get("role")
             if user_role != role:
-                logging.info(f"Access denied for user {session.get('username', 'unknown')} (role: {user_role}) trying to access {role}-only route")
-                flash(f'Access denied. This page is only for {role}s', 'danger')
-                return redirect(url_for('api.index'))
-            
+                logging.info(
+                    f"Access denied for user {session.get('username', 'unknown')} (role: {user_role}) trying to access {role}-only route"
+                )
+                flash(f"Access denied. This page is only for {role}s", "danger")
+                return redirect(url_for("api.index"))
+
             logging.debug(f"Access granted to {session.get('username')} ({user_role}) for {role} route")
             return f(*args, **kwargs)
+
         return decorated_function
+
     return decorator

app/blueprints/__init__.py

@@ -24,7 +24,7 @@
 
 def _apply_no_cache_headers(response: Response) -> Response:
     """Apply no-cache headers to prevent browsers from caching sensitive certificate responses."""
-    response.headers['Cache-Control'] = 'no-store, no-cache, must-revalidate, max-age=0'
-    response.headers['Pragma'] = 'no-cache'
-    response.headers['Expires'] = '0'
+    response.headers["Cache-Control"] = "no-store, no-cache, must-revalidate, max-age=0"
+    response.headers["Pragma"] = "no-cache"
+    response.headers["Expires"] = "0"
     return response