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

add a variable for "mynetworks " in postfix #1064

Closed
WBJUO opened this issue Jul 7, 2021 · 3 comments
Closed

add a variable for "mynetworks " in postfix #1064

WBJUO opened this issue Jul 7, 2021 · 3 comments

Comments

@WBJUO
Copy link

WBJUO commented Jul 7, 2021

Is your feature request related to a problem? Please describe.
Yes each time I add "mynetworks" to /etc/postfix/main.cf it is overwritten on update.
mynetworks = 127.0.0.0/8,192.168.1.10/32,192.168.1.5/32,192.168.1.200/32
This would allow me and others to configure persistent relaying from other machines on our networks.

Describe the solution you'd like
add a variable for "mynetworks" to postfix

Describe alternatives you've considered
I wrote a script to add "mynetworks" to main.cf and restart postfix however when updates happen via cronapt main.cf is overwritten. Adding a variable to OMV config would solve this.
Example
OMV_POSTFIX_MAIN_MYNETWORKS=127.0.0.0/8,192.168.1.10/32,192.168.1.5/32,192.168.1.200/32

Thanks.
William

@votdev
Copy link
Member

votdev commented Jul 8, 2021

It does not make sense to add configuration options (and env variables) for things that do not need to be managed by OMV. Because of that i added a documentation how to add custom Salt states. With this it is easily possible to add custom configuration that is applied whenever OMV rebuilds the configuration.

See https://openmediavault.readthedocs.io/en/5.x/various/advset.html#add-custom-states

@votdev votdev closed this as completed Jul 8, 2021
@mi-hol mi-hol added the reject label Jul 8, 2021
@WBJUO
Copy link
Author

WBJUO commented Jul 8, 2021 via email

@votdev
Copy link
Member

votdev commented Jul 8, 2021

No worries, this was a good opportunity to update the documentation.

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

No branches or pull requests

3 participants