Change script to allow other services restarted as well

[pboosten]

Currently, with every update of anvil I need to modify the script to allow for other services, not explicitly mentioned in the script, to be restarted as well.

Services are (for instance) mattermostd, courier-imap-imapd-ssl, sabnzbd and others.

[dlangille]

What? I have users?

[dlangille]

Yeah, I wonder how best to do this. Sounds like a hook-type thing.

Here, run 'this script'.

How would you prefer to handle it? A configuration item:

USER_RESTART="mattermostd courier-imap-imapd-ssl sabnzbd"
USER_RELOAD="foo bar"

for service in $USER_RESTART
do
  service $service restart
done

for service in $USER_RELOAD
do
  service $service reload
done

[pboosten]

Yeah! And I love it. Cheers, Peter

…

On 30 Oct 2019, at 22:37, Dan Langille ***@***.***> wrote:  What? I have users? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[dlangille]

This is a security risk, but it's all up to the user.

[dlangille]

certpuller -s needs to be updated too.

[pboosten]

This is a security risk, but it's all up to the user.

Not sure why this is a security risk: the script runs as anvil user, and as such it has no authorizations to restart whatever service, unless explicitly configured in sudoers

[dlangille]

I ran this through on one server. Do you want to give it a go?

[pboosten]

Hi Dan, Apologies for late answer: gonna try tonight

…

On 5 Nov 2019, at 14:49, Dan Langille ***@***.***> wrote:  I ran this through on one server. Do you want to give it a go? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[dlangille]

It sounds like it worked.