darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-14 17:22:17 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1838 from GSA/invites_again

Browse Source
This commit is contained in:
Beverly Nguyen
2025-07-16 12:14:24 -07:00
committed by GitHub
parent 64254e5ef7 98b2149cf5
commit c657cfdb2d
2 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
app/user/rest.py
View File

@@ -89,6 +89,7 @@ def handle_integrity_error(exc):
@user_blueprint.route("", methods=["POST"]) @user_blueprint.route("", methods=["POST"])
def create_user(): def create_user():
current_app.logger.info("#invites: enter create_user()")
req_json = request.get_json() req_json = request.get_json()
user_to_create = create_user_schema.load(req_json) user_to_create = create_user_schema.load(req_json)
@@ -96,6 +97,7 @@ def create_user():
user_to_create, password=req_json.get("password"), validated_email_access=True user_to_create, password=req_json.get("password"), validated_email_access=True
) )
result = user_to_create.serialize() result = user_to_create.serialize()
current_app.logger.info("#invites: returning the new user")
return jsonify(data=result), 201 return jsonify(data=result), 201
@@ -597,14 +599,22 @@ def set_permissions(user_id, service_id):
@user_blueprint.route("/get-login-gov-user", methods=["POST"]) @user_blueprint.route("/get-login-gov-user", methods=["POST"])
def get_user_login_gov_user(): def get_user_login_gov_user():
current_app.logger.info("#invites: enter get_user_login_gov_user")
request_args = request.get_json() request_args = request.get_json()
login_uuid = request_args["login_uuid"] login_uuid = request_args["login_uuid"]
email = request_args["email"] email = request_args["email"]
current_app.logger.info(
f"#invites: login_uuid {login_uuid} email is null? {email is None}"
)
user = get_login_gov_user(login_uuid, email) user = get_login_gov_user(login_uuid, email)
if user is None: if user is None:
current_app.logger.info("#invites: couldn't find user, returning empty data")
return jsonify({}) return jsonify({})
current_app.logger.info("#invites: found the user, serializing")
result = user.serialize() result = user.serialize()
current_app.logger.info("#invites: returning successfully")
return jsonify(data=result) return jsonify(data=result)
@@ -615,9 +625,7 @@ def fetch_user_by_email():
hilite(f"enter fetch_user_by_email with {request.get_json()}") hilite(f"enter fetch_user_by_email with {request.get_json()}")
) )
email = email_data_request_schema.load(request.get_json()) email = email_data_request_schema.load(request.get_json())
debug_not_production(hilite(f"request schema loads {email}"))
fetched_user = get_user_by_email(email["email"]) fetched_user = get_user_by_email(email["email"])
debug_not_production(hilite(f"fetched user is {fetched_user}"))
result = fetched_user.serialize() result = fetched_user.serialize()
return jsonify(data=result) return jsonify(data=result)
except Exception as e: except Exception as e:

4
notifications_utils/logging.py
View File

@@ -179,7 +179,7 @@ class JSONFormatter(BaseJSONFormatter):
# for example, delivery receipts, where the formatting fails # for example, delivery receipts, where the formatting fails
# This is not a huge problem, don't dump stack traces into the logs # This is not a huge problem, don't dump stack traces into the logs
# for it. # for it.
logger.warning(f"failed to format log message: {e}") logger.debug(f"failed to format log message: {e}")
except IndexError as e: except IndexError as e:
logger.exception(f"failed to format log message: {e}") logger.debug(f"failed to format log message: {e}")
return log_record return log_record