Skip to content
Snippets Groups Projects

crashfix: SendUserEmailOnFieldUpdate crashes on users without an email

Merged Warren Gifford requested to merge ef/stop-panic-logs into main

Created by: efritz

log15.Log methods have the signature that expects a message, key, value, key, value, .... Here, we're passing ctx where it expects a string, which causes the log line to crash the current goroutine.

This was preventing all e2e tests from running.

Fixes https://github.com/sourcegraph/sourcegraph/issues/14649

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
Please register or sign in to reply
Loading