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

Rfc8621 conformance fix #68

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Rfc8621 conformance fix #68

wants to merge 6 commits into from

Commits on Dec 12, 2023

  1. simplify if-else branches

    @sftse
    sftse committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    ce3add1 View commit details
    Browse the repository at this point in the history

  2. hoist common if condition into outer scope

    @sftse
    sftse committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    adfdc33 View commit details
    Browse the repository at this point in the history

  3. adapt tests

    @sftse
    sftse committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    ca2bce8 View commit details
    Browse the repository at this point in the history

  4. fix: image/*, audio/*, video/* are not attachments when found in both… 

    … text and html parts

RFC8621 section 4.1.4
The additional is_multipart heuristic prevents the tests for malformed .eml
from failing.
    @sftse
    sftse committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    f44cbef View commit details
    Browse the repository at this point in the history

  5. delete unneeded variable

    @sftse
    sftse committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    705c82d View commit details
    Browse the repository at this point in the history

  6. fix clippy warnings

    @sftse
    sftse committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    aef66c0 View commit details
    Browse the repository at this point in the history