Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork323
query: allow <>s around email addresses#4596
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
External address queries should be of the form: name@example.com<tab>Real NameHistorically, (Neo)Mutt would accept email addresses that were wrapped in <>s.This feature stopped working in5d8a416 alias: unify query parsing
When parsing an Alias, NeoMutt stores the "alias" in Alias.name and the"name" and "email" in an Address.Change the Query parsing to store the "name" and "email" the same way.
I don't have a preference, but if it's not supported, would it be easy to at least give an error message so the user knows the issue? I don't think users would mind removing the angle brackets if they know they cause problems. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM
1474f36 intomainUh oh!
There was an error while loading.Please reload this page.
External address queries should be of the form:
Historically, (Neo)Mutt would accept email addresses that were wrapped in
<>s.This feature stopped working in5d8a416 alias: unify query parsing
This PR restores the old (undocumented) behaviour.
Is this what we want?