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.

Sign up for GitHub

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

Jump to bottom

Can not parse email with blank email address headers #159

Closed
pyq opened this issue Aug 15, 2018 · 4 comments
Closed

Can not parse email with blank email address headers #159

pyq opened this issue Aug 15, 2018 · 4 comments
Assignees
@bbottema
Labels
bug Priority-Medium
Milestone
6.0.0-rc1

Comments

@pyq
Copy link

pyq commented Aug 15, 2018 •
edited
Loading

For some incoming emails, it could be with header Return-Path: <>. This is possible for auto-replies emails.
This library always throws MimeMessageParseException when parsing those emails.

related code
@bbottema bbottema added the bug label Aug 16, 2018
@bbottema bbottema self-assigned this Aug 16, 2018
@bbottema bbottema added this to the 6.0.0 milestone Aug 16, 2018
@bbottema
Copy link
Owner

bbottema commented Aug 16, 2018 •
edited
Loading

How do you propose I check if the value is 'empty' -it's actually an invalid value according to the specs.

@bbottema bbottema changed the title Can not parse email with blank return-path. Can not parse email with blank email address headers Aug 16, 2018
bbottema added a commit that referenced this issue Aug 16, 2018
@bbottema
#159: be able to handle empty email header, empty being defined as nu…
05e4163 
…ll, "" and "<>"
@bbottema
Copy link
Owner

bbottema commented Aug 16, 2018 •
edited
Loading

I just implemented a simple empty check which checks for null, "" and "<>". Did this for Return-Path as well as Return-Receipt-To and Disposition-Notification-To.

Will be in release 6.0.0. Thanks for the bug report!

@bbottema
Copy link
Owner

Released in 6.0.0-rc1!

@bbottema
Copy link
Owner

6.0.0 has released as well, finally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bbottema bbottema

Labels
bug Priority-Medium
Projects
None yet
Milestone
6.0.0-rc1
Development

No branches or pull requests
2 participants
@bbottema @pyq