Skip to content

post_authentication overwrites stored email with empty string #162

Description

@dmeiser

Synthesized from full-repo review (KW-REVIEW-SYNTHESIS).

  • Severity: Medium
  • Location: src/handlers/post_authentication.py:83-94
  • Summary: If user_attributes lacks email (phone-only or some social signups), the handler writes email = "", clobbering a previously stored address.
  • Recommendation: Only update email when a non-empty value is present.
  • Source reviews: deepseek-v4-pro

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions