Skip to content

Draft: Improve logging to catch service failures in email validation #2238

Ben requested to merge chore/improve-the-checker-logs-2238 into master

Ticket(s) / Related Merge Requests

Part way to closing #2238 (closed)

Summary of Changes

Errors in TheChecker's email validation service are not being logged correctly in our error tracking software. This means that now we have a failure in #2238 (closed), there's no optics to see what has gone wrong.

Testing Considerations

This is really logging-related thus testing isn't going to be easy. If you want to test you can

kubectl get pods | grep the-checker
kubectl logs [pod from above command]

Try to register with a fake email and watch the logs - note though that no correct key is set on sandbox at the time of writing as this is testing the error state of no key existing, so it will not tell you if you enter a valid email - just that there is an issue with the API key.

Regression Scope

Minor, adds logging to an already broken service.

Platform Affected (web, mobile, etc)

Engine / Sentry

Developer Testing Completed

Manual

Screenshots

(Any extra UI screens that can be shared)

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Merge request reports