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

feature: allocate message_id in mail from: phase #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bjne
Copy link

@bjne bjne commented Mar 2, 2020

Added EXPERIMENTAL_MAILFROM_MSGID to move generation of message_id
from DATA to MAIL FROM phase. This will allow logging of errors
in RCPT phase, referencing a potential future message_id

@HeikoSchlittermann
Copy link
Member

I pulled it sucessfully, I'll run it against my local testsuite first before pushing it to master

@bjne bjne force-pushed the experimental_mailfrom_msgid branch 2 times, most recently from 4a0bd06 to 904ac9d Compare March 5, 2020 18:57
Added EXPERIMENTAL_MAILFROM_MSGID to move generation of message_id
from DATA to MAIL FROM phase. This will allow logging of errors
in RCPT phase, referencing a potential future message_id
@bjne bjne force-pushed the experimental_mailfrom_msgid branch from 904ac9d to 665485e Compare March 5, 2020 19:00
@HeikoSchlittermann
Copy link
Member

He Bjne, after long delay I'm continuing now the integration of your work.

@HeikoSchlittermann
Copy link
Member

rebased it to current master, now I'll check how it passes the testsuite run

@HeikoSchlittermann
Copy link
Member

It still breaks the testsuite run, even if the EXPERIMENTAL_MAILFROM_MSGID is disabled, which, IMHO should not happen

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