Skip to content

Add logging for when lacking session info

Jay requested to merge jswain_add_logging_to_identity_verifications_controller into master

What does this MR do and why?

Add logging to the IdentityVerificationController for when session[:verification_user_id] is missing so we can debug

part of: gitlab-org/modelops/anti-abuse/team-tasks#250

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Monitor the application_json.log: tail -f log/application_json.log
  2. As a logged in user visit http://localhost:3000/users/identity_verification
  3. You should see a new log IdentityVerification::Error with details like: {"severity":"INFO","time":"2023-03-03T18:05:24.012Z","correlation_id":"01GTMACTJ294ZZVRNBVSNRT5HE","message":"IdentityVerification::Error","event":"Verification User Not Found","username":"root","ip":"127.0.0.1","reason":"signed_in: true, state: {\"email\"=\u003etrue}, verified: true","referer":null}

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Jay

Merge request reports