fix: scopes validation logs (#630)

keephq · Dec 13, 2023 · 567d46d · 567d46d · vercel · Dec 13, 2023
1 parent 4c7af79
commit 567d46d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/keep/api/routes/providers.py b/keep/api/routes/providers.py
@@ -302,6 +302,7 @@ def delete_provider(
 def validate_scopes(
     provider: BaseProvider, validate_mandatory=True
 ) -> dict[str, bool | str]:
+    logger.info("Validating provider scopes")
     validated_scopes = provider.validate_scopes()
     if validate_mandatory:
         mandatory_scopes_validated = True
@@ -316,10 +317,17 @@ def validate_scopes(
                     break
         # Otherwise we fail the installation
         if not mandatory_scopes_validated:
+            logger.warning(
+                "Failed to validate mandatory provider scopes",
+                extra={"validated_scopes": validated_scopes},
+            )
             raise HTTPException(
                 status_code=412,
                 detail=validated_scopes,
             )
+    logger.info(
+        "Validated provider scopes", extra={"validated_scopes": validated_scopes}
+    )
     return validated_scopes
 
 

diff --git a/keep/providers/grafana_provider/grafana_provider.py b/keep/providers/grafana_provider/grafana_provider.py
@@ -108,7 +108,7 @@ def validate_scopes(self) -> dict[str, bool | str]:
             f"{self.authentication_config.host}/api/access-control/user/permissions"
         )
         try:
-            response = requests.get(permissions_api, headers=headers).json()
+            response = requests.get(permissions_api, headers=headers, timeout=5).json()
         except requests.exceptions.ConnectionError:
             validated_scopes = {
                 scope.name: "Failed to connect to Grafana. Please check your host."