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

mpick: filter by CC #186

Closed
paper42 opened this issue Sep 25, 2020 · 5 comments
Closed

mpick: filter by CC #186

paper42 opened this issue Sep 25, 2020 · 5 comments

Comments

@paper42
Copy link

paper42 commented Sep 25, 2020

Is this possible? I have a mailing list where sometimes people add the ML to CC, so it doesn't get filtered when I use to =~ "@xxxxxxxx.com".

@Seirdy
Copy link

Seirdy commented Nov 6, 2020

Putting together a tiny patch.

Seirdy added a commit to Seirdy/mblaze that referenced this issue Nov 6, 2020
@Duncaen
Copy link
Collaborator

Duncaen commented Nov 6, 2020

You can already use strings to match any header, the keywords were added at first and are kept for backwards compatibility.

@Duncaen
Copy link
Collaborator

Duncaen commented Nov 6, 2020

"cc" =~ "foo" already works and as far as I understand this issue is about extending the to keyword to not only match the to header, but also the cc header.

@Seirdy
Copy link

Seirdy commented Nov 6, 2020 via email

@paper42
Copy link
Author

paper42 commented Nov 6, 2020

as far as I understand this issue is about extending the to keyword to not only match the to header, but also the cc header.

I will try to phrase my issues more clearly next time. I do not think the "to" keyword should match "To" and "CC" headers, that would be confusing.

I think "cc" ~= ... is enough. I didn't know about it, but now, reading the manpage I noticed this line:

<hdrprop>  ::= from | to | subject | <str>

previously I didn't pay much attention to <str> when I saw From, To and Subject there. I guess there is a deeper reason why these headers are separate from any other header on this line, but it's a bit confusing.

@paper42 paper42 closed this as completed Nov 6, 2020
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 a pull request may close this issue.

3 participants