Skip to content

Handle CTE values case-insensitively#258

Merged
Kludex merged 1 commit intomasterfrom
fix-cte-value-case
Apr 10, 2026
Merged

Handle CTE values case-insensitively#258
Kludex merged 1 commit intomasterfrom
fix-cte-value-case

Conversation

@Kludex
Copy link
Copy Markdown
Owner

@Kludex Kludex commented Apr 10, 2026 •
edited
Loading

Summary

  • normalize Content-Transfer-Encoding values before comparing supported encodings
  • cite RFC 2045 section 6.1 next to the normalization
  • add pytest-parametrized regression coverage for lowercase, uppercase, and mixed-case base64 values

Tests

  • uv run pytest tests/test_multipart.py::test_content_transfer_encoding_is_case_insensitive -q
  • uv run pytest tests/test_multipart.py -q
  • uv run ruff check .

@Kludex Kludex force-pushed the fix-cte-value-case branch from b4e682c to 9832b0d Compare April 10, 2026 11:17
@Kludex Kludex force-pushed the fix-cte-value-case branch from 9832b0d to 74362ed Compare April 10, 2026 11:19
@Kludex Kludex merged commit 9a1ecbd into master Apr 10, 2026
12 checks passed
@Kludex Kludex deleted the fix-cte-value-case branch April 10, 2026 11:21
@Kludex Kludex mentioned this pull request Apr 10, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Kludex