apache / apache/ponymail-foal

Use email.headerregistry classes?

Open
#162 3 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
45
Forks
23
PR merge metrics
No merged PRs in 30d

Description

The headerregistry classes [1] look useful for parsing, especially for address types which are rather complicated.

In particular the AddressHeader handles multiple addresses, and splits the address into display and email.

MessageID was added in 3.8, but does not trim leading spaces. That is trivial to fix.

[1] https://docs.python.org/3.7/library/email.headerregistry.html

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the linked Python email.headerregistry documentation, especially AddressHeader and MessageID, then locate the Pony Mail code that parses address and message-ID headers. Determine which parsing paths should use these classes and verify that leading spaces are removed from MessageID values; the issue names no files or tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.