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

@github-actions

Copy link
Copy Markdown

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

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