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

Remove requirement to run as root #54

Closed
TinyTheBrontosaurus opened this issue Mar 23, 2017 · 3 comments
Closed

Remove requirement to run as root #54

TinyTheBrontosaurus opened this issue Mar 23, 2017 · 3 comments
Labels
enhancement New feature request/PR
Milestone

Comments

@TinyTheBrontosaurus
Copy link

I assume this is possible, but it will be difficult when it comes to mounting. But I get nervous when directories are being mkdir'd and rm'd as root.

@ahnooie
Copy link
Member

ahnooie commented Mar 29, 2017

Agreed, definitely want to get it working as non-root soon. The ARM could either su to an underprivileged user for most tasks, or sudo to root only when it needs it.

@ahnooie ahnooie added this to the v1.6.0 milestone Jul 29, 2017
@ahnooie ahnooie added the enhancement New feature request/PR label Jul 29, 2017
@2long1
Copy link

2long1 commented Dec 12, 2017

Adding User=foo to the arm@.service file would cause the identify script to be be run as the user foo. With a judicious use of sudo, you could probably drastically reduce what runs as root.

@ahnooie ahnooie modified the milestones: v1.6.0, v2.0.0 Feb 1, 2018
@muckngrind4
Copy link
Collaborator

Added feature in commit d3b9c6d

flammableliquids pushed a commit that referenced this issue Jun 9, 2021
suggestions 1,2 from issue #54 now added

Small changes to models.py to remove some code duplication.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature request/PR
Projects
None yet
Development

No branches or pull requests

4 participants