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

New email with another one as attachment is considered as bounce #296

Closed
whity opened this issue Nov 8, 2018 · 8 comments
Closed

New email with another one as attachment is considered as bounce #296

whity opened this issue Nov 8, 2018 · 8 comments
Labels

Comments

@whity
Copy link

whity commented Nov 8, 2018

Hello,

Found an issue that seems like a bug.
Having an email with other email as attachment, the sisimai is considering it as a bounce.
Could you please take look, thanks.

email.txt
perl-dump.txt

@azumakuniyuki
Copy link
Member

Hello,

Thanks for the report. I've confirmed that sisimai detects the email as a bounce. It may be a bug in Sisimai::RFC3464 class.

Thanks :-)

azumakuniyuki added a commit that referenced this issue Nov 10, 2018
azumakuniyuki added a commit to sisimai/rb-sisimai that referenced this issue Nov 10, 2018
azumakuniyuki added a commit to sisimai/rb-sisimai that referenced this issue Nov 10, 2018
azumakuniyuki added a commit to sisimai/rb-sisimai that referenced this issue Nov 10, 2018
@whity
Copy link
Author

whity commented Nov 10, 2018

Hi @azumakuniyuki , many thanks for the quick fix.

@azumakuniyuki
Copy link
Member

@whity Hello, I've fixed this bug and registered the email you attached above as set-of-emails/maildir/not/is-not-bounce-02.eml for using make test at p5-Sisimai, rb-Sisimai, and set-of-emails repository.

Thanks :-)

@azumakuniyuki
Copy link
Member

Fixed code is included v4.24.0p1 tag: https://github.com/sisimai/p5-Sisimai/releases/tag/v4.24.0p1

@whity
Copy link
Author

whity commented Nov 12, 2018

Hi @azumakuniyuki , just a quick question.
There will be a patch release, e.g. 'v4.24.1'?

Thank you.

@azumakuniyuki
Copy link
Member

@whity Hello, Sisimai 4.24.1 (including this fix and other bug fixes) will be released on 14 Nov.

@azumakuniyuki
Copy link
Member

@whity We've released v4.24.1 just now :-)

@whity
Copy link
Author

whity commented Nov 14, 2018

@azumakuniyuki cool , thanks.

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

No branches or pull requests

2 participants