Skip to content
This repository has been archived by the owner on Jan 27, 2024. It is now read-only.

Trailing spaces in subjects in MailScanner trigger duplicate subjects in header #219

Closed
shawniverson opened this issue Nov 4, 2015 · 2 comments
Assignees
Labels
Milestone

Comments

@shawniverson
Copy link
Member

Test against latest MailScanner code.

If confirmed, fix it.

https://forum.efa-project.org/viewtopic.php?f=13&t=1294

@shawniverson shawniverson added this to the 3.0.0.9 milestone Nov 4, 2015
@shawniverson shawniverson changed the title Trailing spaces in MailScanner trigger duplicate subjects in header Trailing spaces in subjects in MailScanner trigger duplicate subjects in header Nov 4, 2015
@shawniverson shawniverson self-assigned this Nov 21, 2015
@shawniverson
Copy link
Member Author

From MailScanner List...

Hmm..

My version of mailscanner on efa 3.0.0.8 is 4.84.6

My MailScanner settings are​:
Multiple Headers = add
Place New Headers At Top Of Message = yes

I vaguely remember changing the Multiple Headers setting from append to add because of dkim. Unfortunately, I might be misremembering. I cannot remember and I didn't keep notes. All I am sure of it that it solved another problem for me.

  • Paul

---------- Forwarded message ----------
From: Kevin Miller kevin.miller@juneau.org
To: "'MailScanner Discussion'" mailscanner@lists.mailscanner.info
Cc:
Date: Wed, 4 Nov 2015 18:09:47 +0000
Subject: RE: duplicate subject lines causing yahoo mail rejection
What’s an “efa”?

I was having the same issue earlier this year. I don’t know if it was due to spaces in the subject, but we were getting rejects from yahoo due to multiple subjects injected by MailScanner.

The workaround (I hesitate to call it a solution) was posted by Scott Anderson on 2/23, subject “RE: DKIM and MailScanner Watermarking”:
Multiple Headers = append

Place New Headers At Top Of Message = yes

...Kevin

MailScanner mailing list
mailscanner@lists.mailscanner.info
http://lists.mailscanner.info/listinfo/mailscanner

shawniverson added a commit that referenced this issue Nov 21, 2015
@shawniverson
Copy link
Member Author

Fixed in commit 25e97c7

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

No branches or pull requests

1 participant