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

fix: set permission when postinst #386

Merged
merged 9 commits into from
Sep 15, 2024
Merged

Conversation

da13da
Copy link
Contributor

@da13da da13da commented Sep 12, 2024

I made the following changes:

  • The agent is started by nodex via systemd, which prevented other users from accessing the socket. I updated the permissions to allow access.
  • Updating binaries in /opt/bin requires sudo privileges, so I moved the binary to the /home/nodex directory for easier future updates.
root@902092186ed4:/nodex# useradd -r -s /bin/false test
root@902092186ed4:/nodex# sudo -u test curl --unix-socket /home/nodex/.nodex/run/nodex.sock http://localhost/internal/version/get
{"version":"3.0.0"}root@902092186ed4:/nodex# 

@da13da da13da marked this pull request as ready for review September 13, 2024 04:48
@da13da da13da requested a review from a team as a code owner September 13, 2024 04:49
@da13da da13da requested a review from curry-like September 13, 2024 04:49
@da13da da13da merged commit f7878d3 into main Sep 15, 2024
14 checks passed
@da13da da13da deleted the fix/systemd-and-runtime-path branch September 15, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants