Skip to content

fix: more stringent checks on valid email addresses while autocompleting recipients#13212

Open
madbob wants to merge 1 commit into
nextcloud:mainfrom
madbob:issue_10432
Open

fix: more stringent checks on valid email addresses while autocompleting recipients#13212
madbob wants to merge 1 commit into
nextcloud:mainfrom
madbob:issue_10432

Conversation

@madbob

@madbob madbob commented Jul 4, 2026

Copy link
Copy Markdown
Collaborator

To avoid duplications I've reused the isValidEmail() function took from AccountForm.vue and moved in the global utilities file emailAddress.js

Fixes #10432

…ing recipients

Signed-off-by: Roberto Guido <info@madbob.org>
@ChristophWurst

Copy link
Copy Markdown
Member

Thanks for the PR

@ChristophWurst

Copy link
Copy Markdown
Member

I tried the STR from #10432 but can not reproduce on main.

How can we test this?

@madbob

madbob commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator Author
  • have a Contact with an email (e.g. foobar@example.com)
  • in Mail, start a new message
  • in the To field, start writing "foobar@ex". The fact to add @ and one or more chars for the domain is relevant
  • select the contact from the autocomplete list: this will add "foobar@example.com" to the recipients list
  • blur the To field
  • at this point, to the To field is appended another recipient having as email address "foobar@ex" (the partial address you were writing)
Screencast.From.2026-07-16.14-58-20.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Contact search with @ is added to recipients, blocking outbound

2 participants