add TEMP logging for base user creation, login 2.61

Author: c-simpson

src/helm-chart/values.yaml

@@ -9,14 +9,14 @@ server:
     repository: ghcr.io/codeforphilly/paws-data-pipeline/server
     pullPolicy: Always
     # Overrides the image tag whose default is the chart appVersion.
-    tag: "2.60"
+    tag: "2.61"
 
 client:
   image:
     repository: ghcr.io/codeforphilly/paws-data-pipeline/client
     pullPolicy: Always
     # Overrides the image tag whose default is the chart appVersion.
-    tag: "2.60"
+    tag: "2.61"
 
 db:
   image:

src/server/api/user_api.py

@@ -135,15 +135,18 @@ def dummy_check():
         check_password('password', pw_bytes)
 
     try:
+        logger.debug(request.data)  #TODO: Remove
         post_dict = json.loads(request.data)
         username = post_dict["username"]
         presentedpw = post_dict["password"]
-    except:
+    except Exception as err:
         dummy_check()    # Take the same time as with well-formed requests 
+        logger.info("Exception parsing or getting username/pw : %s", str(err))
         return jsonify("Bad credentials"), 401
 
     if not (isinstance(username, str) and isinstance(presentedpw, str) ):
         dummy_check()  # Take the same time as with well-formed requests 
+        logger.info("Not string")
         return jsonify("Bad credentials"), 401   # Don't give us ints, arrays, etc.
 
 
@@ -163,6 +166,7 @@ def dummy_check():
             pwhash, role, is_active = result.fetchone()
         else:
             log_user_action(username, "Failure", "Invalid username")
+            logger.info("Unknown username")
             dummy_check()
             return jsonify("Bad credentials"), 401
 
@@ -175,6 +179,7 @@ def dummy_check():
         else:
             log_user_action(username, "Failure", "Bad password or inactive")
             # No dummy_check needed as we ran a real one to get here
+            logger.info("Bad password or inactive user")
             return jsonify("Bad credentials"), 401
 
 

src/server/db_setup/base_users.py

@@ -76,6 +76,7 @@ def create_base_users():  # TODO: Just call create_user for each
             ins_stmt = pu.insert().values(
                 username="base_admin", full_name="Base Admin", password=pw_hash, active="Y", role=9,
             )
+            logger.info("base_admin password: %s", BASEADMIN_PW) #TODO: Remove this
             connection.execute(ins_stmt)
 
         else: