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

[Feat]: Autoreload htpasswd on changes #2925

Open
vooon opened this issue Jan 30, 2025 · 2 comments · May be fixed by #2933
Open

[Feat]: Autoreload htpasswd on changes #2925

vooon opened this issue Jan 30, 2025 · 2 comments · May be fixed by #2933
Labels
feature New feature or request rm-external Roadmap item submitted by non-maintainers

Comments

@vooon
Copy link
Contributor

vooon commented Jan 30, 2025

Is your feature request related to a problem? Please describe.

In my case part of accounts managed by external system, which may dynamically update htpasswd files.
That works well with the Nginx, but not so well with Zot as it do not reload the file.

That can be done by accepting reload signal (like HUP), or by Inotify.
In case of HUP i'd prefer the node also reload TLS certs, so things like acme-redirect don't need to do a full restart.
But cert rotation is a much less frequent and can be configured. While temporary accounts - cant.

Describe the solution you'd like

Watch for Inotify events on Linux and reload on close event.

Describe alternatives you've considered

No response

Additional context

No response

@vooon vooon added the feature New feature or request label Jan 30, 2025
@rchincha rchincha added the rm-external Roadmap item submitted by non-maintainers label Jan 30, 2025
@rchincha
Copy link
Contributor

We have this enabled for some aspects of the configuration (authorization, etc)
Pls. feel free to post a PR.

@andaaron
Copy link
Contributor

We have similar logic for the ldap config file, see #2167

@vooon vooon linked a pull request Jan 31, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request rm-external Roadmap item submitted by non-maintainers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants