Add more debug logs

integration_tests/subroutes/di_subrouter.py

@@ -8,9 +8,9 @@
 
 
 @di_subrouter.get("/subrouter_router_di")
-def sync_subrouter_route_dependency(request, router_dependencies):
+def sync_subrouter_route_dependency(router_dependencies):
     return router_dependencies["ROUTER_DEPENDENCY"]
 
 @di_subrouter.get("/subrouter_global_di")
-def sync_subrouter_global_dependency(request, global_dependencies):
+def sync_subrouter_global_dependency(global_dependencies):
     return global_dependencies["GLOBAL_DEPENDENCY"]

robyn/dependency_injection.py

@@ -1,12 +1,12 @@
 """ This is Robyn's dependency injection file.
 """
+from typing import Any
 
 class DependencyMap:
     def __init__(self):
-        #'request' and 'response' mappings are needed for when constructing deps_to_pass in router.py
-        self.global_dependency_map = {}
+        self.global_dependency_map: dict[str, Any] = {}
         # {'router': {'dependency_name': dependency_class}
-        self.router_dependency_map = {}
+        self.router_dependency_map: dict[str, dict[str, Any]] = {}
 
     def add_router_dependency(self, router, **kwargs):
         """Adds a dependency to a route.

robyn/router.py

@@ -97,11 +97,8 @@ def add_route(
         exception_handler: Optional[Callable],
         injected_dependencies: dict,
     ) -> Union[Callable, CoroutineType]:
-        # this should not be an unordered map. should be a list only
         @wraps(handler)
         async def async_inner_handler(*args, **kwargs):
-            print("This is the args", args)
-            print("This is the kwargs", kwargs)
             try:
                 response = self._format_response(
                     await handler(*args, **kwargs),
@@ -117,7 +114,6 @@ async def async_inner_handler(*args, **kwargs):
         @wraps(handler)
         def inner_handler(*args, **kwargs):
             try:
-                # do we even need this?
                 response = self._format_response(
                     handler(*args, **kwargs),
                 )
@@ -140,7 +136,7 @@ def inner_handler(*args, **kwargs):
             if dependency in params:
                 new_injected_dependencies[dependency] = injected_dependencies[dependency]
             else:
-                _logger.warning(f"Dependency {dependency} is not used in the handler {handler.__name__}")
+                _logger.debug(f"Dependency {dependency} is not used in the handler {handler.__name__}")
 
 
         if iscoroutinefunction(handler):
@@ -179,9 +175,7 @@ def add_route(self, middleware_type: MiddlewareType, endpoint: str, handler: Cal
             if dependency in params:
                 new_injected_dependencies[dependency] = injected_dependencies[dependency]
             else:
-                logging.warning(f"Dependency {dependency} is not used in the middleware handler {handler.__name__}")
-
-        print("This is new injected dependencies", new_injected_dependencies)
+                _logger.debug(f"Dependency {dependency} is not used in the middleware handler {handler.__name__}")
 
 
 

robyn/ws.py

@@ -44,7 +44,7 @@ def inner(handler):
                 if dependency in params:
                     new_injected_dependencies[dependency] = injected_dependencies[dependency]
                 else:
-                    _logger.warning(f"Dependency {dependency} is not used in the handler {handler.__name__}")
+                    _logger.debug(f"Dependency {dependency} is not used in the handler {handler.__name__}")
 
                 self.methods[type] = FunctionInfo(handler, is_async, num_params, params, kwargs=new_injected_dependencies)
                 self.robyn_object.add_web_socket(self.endpoint, self)