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

Fixed #59 #60

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fixed #59 #60

wants to merge 3 commits into from

Conversation

3kami3
Copy link

@3kami3 3kami3 commented Apr 3, 2024

Fixed #59.

@Fmstrat
Copy link
Owner

Fmstrat commented Apr 3, 2024

Thank you for the PR. Will this cause issues for people already running as root on upgrade with file permissions? If so, we will need to include a check, or allow supplying of a user on the ENV with default to root so it's not a breaking change.

@3kami3
Copy link
Author

3kami3 commented Apr 4, 2024

Thank you for the PR. Will this cause issues for people already running as root on upgrade with file permissions? If so, we will need to include a check, or allow supplying of a user on the ENV with default to root so it's not a breaking change.

Errors and warnings are output in the current latest version, but nothing is specified, so the supervisord process is already running with root privileges.
If I add user=root to the configuration file, the errors and warnings do not occur. I see no effect on file permissions.

@Fmstrat
Copy link
Owner

Fmstrat commented Apr 5, 2024

Sounds good. Why the addition of the insecure http server? For that to be merged we'd need a good case for it, and it would need to be an option via env along with the credentials.

@3kami3
Copy link
Author

3kami3 commented Apr 8, 2024

Sounds good. Why the addition of the insecure http server? For that to be merged we'd need a good case for it, and it would need to be an option via env along with the credentials.

The reason for adding the unix_http_server section is
"CRIT Server 'unix_http_server' running without any HTTP authentication checking".
The solution is based on the following supervisor issue
Supervisor/supervisor#694
Supervisor/supervisor#717 (comment)

kapdap added a commit to kapdap/samba-domain that referenced this pull request Nov 13, 2024
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.

Bug: Supervisord Error and Warning
2 participants