Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mail Rules matches name, not email. #1110

Closed
KristerV opened this issue Oct 4, 2018 · 3 comments
Closed

Mail Rules matches name, not email. #1110

KristerV opened this issue Oct 4, 2018 · 3 comments

Comments

@KristerV
Copy link

KristerV commented Oct 4, 2018

Are there any related issues? no
What operating system are you using? Arch Linux
What version of Mailspring are you using? 1.4.2
Bug
Is this an email sync or connection issue? no
Is the issue related to a specific email provider (Gmail, Exchange, etc.)? no

What I'm trying to do

I have a @krister.ee hosted by mailbox.org. If I get an email it's addressed to either of the 3:

"Anything" means I've got a catch-all set up.

So I want to create a rule that moves any email not containing krister.ee as a recipient to the spam folder.

How I did it

I create a mail rule and use "Process entire mailbox" to see results:

image

What I expected

What actually happened

What else I've tried

  • Searching the knowledgebase
  • Googling for any tips
  • Using "Macthes expression" with negative lookahead regex (btw, there is NO info whatsoever if this field is actually for regex)
  • Other combinations of rules

What I think is the bug

Seems to me like the rule matches correctly only if the To field has no name (only email). If there is a Name Infront <of@email.com> then the name is matched, but not email..

@KristerV KristerV changed the title Mail Rules To field only affects name Mail Rules matches name, not email. Oct 4, 2018
@bengotow
Copy link
Collaborator

bengotow commented Nov 4, 2018

Hey! Thanks for filing this—I dug in to this today and it looks like you're definitely right. I think the does not contain expression is currently implemented as any of the To components does not contain , when it should read all of the To components do not contain. So if a name is present, and the name doesn't match while the email does, it returns true.

I'll get this fixed today. 👍 Thanks for the detailed bug report!

@KristerV
Copy link
Author

KristerV commented Nov 5, 2018

Awesome, thanks :)

@bengotow
Copy link
Collaborator

Hey! The fix for this shipped in v1.5.4 this afternoon. Mailspring should auto-update in the next few hours and you can also download the update directly from https://getmailspring.com/download. Happy new years!

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

No branches or pull requests

2 participants