Merge branch 'dev'

Author: akhil27051999

app/__init__.py

@@ -8,40 +8,40 @@
 
 def create_app(config_class=None):
     app = Flask(__name__)
-
-    # Configure the app
+    
     if config_class:
         app.config.from_object(config_class)
     else:
         app.config.from_object('app.config.Config')
+    
+    # Set up the dedicated student logger
+    student_logger = setup_logger()
+    
+    # Attach handlers and level to Flask's native logger to ensure all logs write to the same file
+    app.logger.handlers = student_logger.handlers[:]
+    app.logger.setLevel(student_logger.level)
+    app.logger.propagate = False  # Prevent double logging
 
-
-    # Initialize logger
-    logger = setup_logger('student_logger', log_file='app.log')
     app.logger.info("Student Management API started successfully.")
 
-    # Initialize database and migration
+    # init db & migrations
     db.init_app(app)
     migrate.init_app(app, db)
 
-    # Import models inside app context
     with app.app_context():
         from . import models
-
-    # Import and register blueprints here (after db/models are ready)
+    
     from .routes import student_bp
     app.register_blueprint(student_bp, url_prefix='/students')
-
-    # Home route
+    
     @app.route('/')
     def home():
         app.logger.info("Home page accessed")
         return "Welcome to the Student Management API!"
-
-    # Health check route
+    
     @app.route('/health')
     def health():
         app.logger.info("Health check accessed")
         return jsonify({"status": "ok"}), 200
-
-    return app
+    
+    return app

app/logger.py

@@ -1,22 +1,27 @@
 import logging
 import os
 
-def setup_logger(name, log_file='app.log', level=logging.DEBUG):
+def setup_logger(name='student_logger', log_file=None, level=logging.DEBUG):
+    if log_file is None:
+        # Use absolute path based on your repo root
+        repo_root = os.path.abspath(os.path.join(os.path.dirname(__file__), '..'))
+        log_file = os.path.join(repo_root, 'student_api.log')
 
+    # Create logger with specified name
     logger = logging.getLogger(name)
     logger.setLevel(level)
-
-    # Avoid adding multiple handlers if logger already exists
+    
+    # Avoid multiple handlers added on repeated calls
     if not logger.hasHandlers():
         # Console handler
         console_handler = logging.StreamHandler()
         console_handler.setLevel(level)
         console_formatter = logging.Formatter(
             '%(asctime)s - %(name)s - %(levelname)s - %(message)s'
         )
-        console_handler.setFormatter(console_formatter) 
-        logger.addHandler(console_handler) 
-
+        console_handler.setFormatter(console_formatter)
+        logger.addHandler(console_handler)
+        
         # File handler
         file_handler = logging.FileHandler(log_file)
         file_handler.setLevel(level)
@@ -25,5 +30,5 @@ def setup_logger(name, log_file='app.log', level=logging.DEBUG):
         )
         file_handler.setFormatter(file_formatter)
         logger.addHandler(file_handler)
-
-    return logger
+        
+    return logger