Skip to content

Conversation

@dschom
Copy link
Contributor

@dschom dschom commented Jan 29, 2026

Because

  • Account reset was failing in admin panel
  • This in turn resulted in a DB error on insert
  • The hunch is we are getting an set of IPs separated by commas.

This pull request

  • Sanitizes the IP before recording it in security events.

Issue that this pull request solves

Closes: FXA-12978

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

Any other information that is important to this pull request.

@dschom dschom requested a review from a team as a code owner January 29, 2026 05:34
@dschom dschom merged commit 4341661 into main Jan 29, 2026
21 checks passed
@dschom dschom deleted the fix-acct-reset-in-admin-panel branch January 29, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants