Skip to content

feat: add authenticated API to forcibly request configuration reload #1068

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

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

Conversation

madpah
Copy link

@madpah madpah commented Apr 8, 2025

Summary

Further to discussions on #393, and #1064, I'm offering this PR (as a starting point) to allow an external request to be made to POST /v1/config/reload to request configuration is reloaded.

This will allow for external systems to request configuration reload where the current file monitoring either:

  1. Does not detect file modification dates/times as expected
  2. Configuration files are removed, and thus do not trigger the current configuration modification detection logic

If there is appetite for this PR to be merged and accepted, I'll complete the below checklist @TwiN.

Checklist

  • Tested and/or added tests to validate that the changes work as intended, if applicable.
  • Updated documentation in README.md, if applicable.

@github-actions github-actions bot added the feature New feature or request label Apr 8, 2025
@madpah madpah marked this pull request as ready for review April 9, 2025 07:58
@madpah
Copy link
Author

madpah commented Apr 14, 2025

@TwiN - any appetite to accept this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant