Add logging to confidant ingress (#390)

Ingress sometimes is being affected by 4xx's, this will add more observability for when this issue happens. --------- Co-authored-by: Dean Liu <[email protected]>
lyft · Jul 5, 2023 · 85a6676 · 85a6676
1 parent cd54667
commit 85a6676
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/confidant/routes/services.py b/confidant/routes/services.py
@@ -259,6 +259,7 @@ def get_service(id):
             id
         )
         error_msg = {'error': msg, 'reference': id}
+        logger.warning(msg)
         return jsonify(error_msg), 403
 
     logger.info(
@@ -277,9 +278,14 @@ def get_service(id):
             msg = 'Authenticated user is not authorized.'
             return jsonify({'error': msg}), 401
     except DoesNotExist:
+        logger.warning('Item with id {0} does not exist.'.format(id))
         return jsonify({}), 404
+    except Exception as e:
+        logger.warning('Exception occurred: {0}'.format(e))
+        raise e
     if (service.data_type != 'service' and
             service.data_type != 'archive-service'):
+        logger.warning('Item with id {0} is not a service.'.format(id))
         return jsonify({}), 404
     logger.debug('Authz succeeded for service {0}.'.format(id))
     try: