more precise generic healthcheck

api_app/classes.py

@@ -391,13 +391,20 @@ def health_check(self, user: User = None) -> bool:
                 # momentarily set this to False to
                 # avoid fails for https services
                 response = requests.head(url, timeout=10, verify=False)
+                # This may happen when even the HEAD request is protected by authentication
+                # We cannot create a generic health check that consider auth too
+                # because every analyzer has its own way to authenticate
+                # So, in this case, we will consider it as check passed because we got an answer
+                # For ex 405 code is when HEADs are not allowed. But it is the same. The service answered.
+                if 400 <= response.status_code <= 408:
+                    return True
                 response.raise_for_status()
             except (
                 requests.exceptions.ConnectionError,
                 requests.exceptions.Timeout,
                 requests.exceptions.HTTPError,
             ) as e:
-                logger.info(f"healthcheck failed: url {url}" f" for {self}. Error: {e}")
+                logger.info(f"healthcheck failed: url {url} for {self}. Error: {e}")
                 return False
             else:
                 return True