Skip to content

fix: don't allow email aliases on registration - #573

Open
ciurana-life wants to merge 1 commit into
inveniosoftware:masterfrom
ciurana-life:fix-security-html-injection
Open

fix: don't allow email aliases on registration#573
ciurana-life wants to merge 1 commit into
inveniosoftware:masterfrom
ciurana-life:fix-security-html-injection

Conversation

@ciurana-life

@ciurana-life ciurana-life commented Aug 28, 2026

Copy link
Copy Markdown

Description

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Frontend

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.

* It was possible to re-send the account creation email
to a registered user using an alias

* Combined with injecting a url on the email this was a problem

* No impact on account creation since no email provider allows
normal accounts to contain "+" when creating them

Signed-off-by: Victor Ciurana <victor.ciurana.loeffler@cern.ch>
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.

1 participant