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
qmail-smtpd: introduce an array of EHLO string generators #172
base: master
Are you sure you want to change the base?
Conversation
Doesn't simply adding a newline to |
No, it doesn't. STARTTLS introduces a |
But what's the difference between:
and
|
Right, that would be the same. But it would be possible even now, and it has not happened yet. So I guess we need to invent some sort of infrastructure, the crappy it may be, to move people in the right direction. |
a09a08c
to
e75d3c4
Compare
The use case we might want to get hanlded (maybe the most complex): starttls and authentication: Auth advertized never on port 25, never on non-tls, always on 465, only after STARTTLS on port 587. But even so, I do not see a technical advantage on a function pointer, as then, there are still patch(1) conflicts that can occur: patch only sees text. Do we want to move from patch toward (statically linked) modules interacting with qmail at pre-defined hooks? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am happy with it, two suggestions that might be for another commit.
Related, I've written up some design ideas for TLS, AUTH, etc.. Maybe this gives us a bigger picture worth arguing about :-) |
This is of no direct use for the vanilla version, but it allows to simplify the integration of different patches which add add EHLO arguments. They will only need to add their print function to this array and it will get called.
This is of no direct use for the vanilla version, but it allows to simplify the integration of different patches which add add EHLO arguments. They will only need to add their print function to this array and it will get called.