BE-572: User email management improvements#8735
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Email verification is now treated as an explicit authorization signal: org invitation accept/decline checks match against Separately, Reviewed by Cursor Bugbot for commit 03ad748. Bugbot is set up for automated code reviews on this repo. Configure here. |
🤖 Augment PR SummarySummary: This PR hardens user email management by tightening how email verification is used for authorization-sensitive flows and by preventing direct mutation of Kratos identity traits. Changes:
Technical Notes: Email verification is now treated as an authorization signal for certain actions (e.g., org invite flows), and email trait reads are centralized around Kratos verifiable addresses. 🤖 Was this summary useful? React with 👍 or 👎 |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #8735 +/- ##
==========================================
- Coverage 62.72% 62.71% -0.02%
==========================================
Files 1363 1363
Lines 139062 139087 +25
Branches 5818 5825 +7
==========================================
Hits 87225 87225
- Misses 50921 50946 +25
Partials 916 916
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
… to issue org invite to it
See the output of git range-diff at https://github.com/hashintel/hash/actions/runs/26175332934
20550c5 to
8974830
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 8974830. Configure here.
See the output of git range-diff at https://github.com/hashintel/hash/actions/runs/26176310603

🌟 What is the purpose of this PR?
Security hardening:
Plus cleaning up some function arguments that weren't being used.
Pre-Merge Checklist 🚀
🚢 Has this modified a publishable library?
This PR:
📜 Does this require a change to the docs?
The changes in this PR:
🕸️ Does this require a change to the Turbo Graph?
The changes in this PR:
🛡 What tests cover this?