-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
Run TRSS server as non-root #1327
Comments
Did this get solved during the recent restart of TRSS? |
Unlikely IMHO |
What alternatives are you aware of, can systemd or something else replace this functionality (that forever service provides)? |
Perhaps, although systemd also runs as root. I really need someone to convince me of why forever isn't an option - I've only heard anecdotal reasons so far :-) I've just tested a basic |
@llxia - were there other reasons that you are aware of (re: #1327 (comment))? |
When running the forever-service commands as a non root user, I get an error asking for them to be run as root |
To summarise a lengthy slack thread, I found that the TestResultSummaryService can be started and stopped with the
Both are commands to start up the service, and not the client. Concerning the react client, I am not sure where on the server it is running, so I do not yet know if that can be stopped and started by a non root user |
No idea how |
I am not sure what was changed in your experiment. From TRSS perspective, we do not need to "start" the client. It is a static pre-compiled web client. IMO, the key problem here is that we need root permission to start a service (i.e., forever-service in this case). And we need the ability to restart the service to pick up new code changes. I think we can use |
That clears things up. Thanks |
according to usage: https://github.com/zapty/forever-service/blame/master/README.md#L117 I think there are some misunderstanding in the above comments:
|
Thanks @zdtsw for investigating this. Anyway, we already have a solution and it is up and running internally. We are using the |
I believe this is complete. If anyone disagrees or believes additional work is required then it can be reopened. |
At the moment (discovered while looking at request #1326) the TRSS server is running as root through
forever
and started by theTRSS*
scripts in/etc/init.d
. Running such services as root unnecessarily is undesirable from a security standpoint so we should look at reworking this to run as a non-root user and manage access accordingly.The text was updated successfully, but these errors were encountered: