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

Bug fix and improvement 2021 Nov #442

Merged
merged 7 commits into from
Nov 19, 2021

Conversation

azumakuniyuki
Copy link
Member

  • Fix serious bugs:
    • Values of deliverystatus and replycode detected from the message body did not use at Sisimai::Lhost::Exim.
    • true() method strictly checks the value of smtpcommand at some classes in Sisimai::Reason. For example, when a detected reason is spamdetected and virusdetected the value of smtpcommand should be DATA or an SMTP command to be sent after DATA.

@@ -50,6 +50,13 @@ v4.25.11p2
Thanks to @vhenon
- Add `rfc3464-41.eml` and `rfc3464-42.eml`
- Remove all the HTML elements from the value of "diagnosticcode".
- Fix a serious bugs:
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix serious bugs:

@@ -50,6 +50,13 @@ v4.25.11p2
Thanks to @vhenon
- Add `rfc3464-41.eml` and `rfc3464-42.eml`
- Remove all the HTML elements from the value of "diagnosticcode".
- Fix a serious bugs:
- Values of `deliverystatus` and `replycode` detected from the message body
did not used at `Sisimai::Lhost::Exim`.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did not use

azumakuniyuki added a commit that referenced this pull request Nov 19, 2021
azumakuniyuki added a commit that referenced this pull request Nov 19, 2021
azumakuniyuki added a commit that referenced this pull request Nov 19, 2021
@azumakuniyuki azumakuniyuki deleted the bugfix-and-improvement-2021-nov branch May 16, 2022 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant