-
Notifications
You must be signed in to change notification settings - Fork 100
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
List parameter "host" should be abolished #43
Comments
So indeed, implementation is harder with this parameter but it has its use cases. Before removing this parameter, we should think about whether replacements exist for the current instances using it. |
ikedas
added a commit
to ikedas/sympa
that referenced
this issue
Nov 8, 2017
… value of ``host`` parameter of the list, not mail domain: If ``host`` parameter was not the same as domain name, external program could fail. Fixed by feeding mail domain instead of ``host`` value. And added [% list.host %] template variable to use ``host`` value in template. Note that ``host`` parameter may be deprecated in the future (See issue sympa-community#43).
PR above will add notice on deprecation during upgrading process. |
ikedas
added a commit
that referenced
this issue
Feb 24, 2018
Issue #43: Preliminary notice on abolishment of "host" list parameter by ikedas.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Lists can have
host
parameter overriding host part of list addresses derived from virtual host the list belongs to.Cons:
It breaks alias management: When a list is removed, alias manager cannot find aliases to be removed; If users (owners) change this parameter, aliases won't be renamed. At least current implementaion cannot handle those situations properly.
Pros:
Users (listmasters) can categorize lists with multiple domains under one virtual host. It may be useful for some service providors hosting list services to users by each virtual domain.
IMO this feature would be better to be removed for simplicity of implementation.
The text was updated successfully, but these errors were encountered: