-
Notifications
You must be signed in to change notification settings - Fork 531
chore(config): add MetricsWithoutAuth #906
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
calmh
reviewed
Apr 13, 2025
calmh
approved these changes
Apr 13, 2025
calmh
approved these changes
Apr 13, 2025
calmh
pushed a commit
to syncthing/syncthing
that referenced
this pull request
Apr 13, 2025
### Purpose Give the ability to skip authentication for prometheus metrics ("/metrics"). ### Testing When authentication is enabled and "Metrics Without Auth" is checked (not the default), the "/metrics" path remains accessible even when disconnected. ### Screenshots  ### Documentation syncthing/docs#906
marbens-arch
added a commit
to marbens-arch/syncthing-docs
that referenced
this pull request
Apr 14, 2025
* chore(contrib): move syncthing-android and SyncTrayzor to old list (syncthing#895) SyncTrayzor hasn't received any commits in 4 years, and syncthing-android is officially discontinued, so add them to the old list. Fixes syncthing#892. * chore(contrib): stop calling Syncthing-Fork an "alternative" (syncthing#896) Co-authored-by: André Colomb <[email protected]> * Update author list * fix: word order in ignoring.rst (syncthing#897) * Update author list * fix(faq): update info about checking if a device is relaying (syncthing#899) Devices now still show "Address", even if connected via a relay, so we change the information to still be accurate. The "address" that it shows refers to the address of the relay, if connected via a relay. * Update author list * Update version list * chore(config): simplify list of STUN servers (syncthing#902) We're removing discontinued STUN servers in syncthing/syncthing#10012, so we update the documentation accordingly, while simplifying. * fix(config): update default `stunKeepaliveStartS` (syncthing#903) Ever since syncthing/syncthing@e714df0, `stunKeepaliveStartS` has defaulted to 180, and not 24. * Update version list * Update author list * fix(strings): differentiate setup(n) and set(v) up (syncthing#905) * Update author list * chore(config): add MetricsWithoutAuth (syncthing#906) syncthing/syncthing#10045 * fix(relay-v1): state that both A and B are in session mode (fixes syncthing#802) Permanent protocol and temporary protocol submodes don't exist in session mode. --------- Co-authored-by: Pander <[email protected]> Co-authored-by: André Colomb <[email protected]> Co-authored-by: Syncthing Release Automation <[email protected]> Co-authored-by: Wojciech Geisler <[email protected]> Co-authored-by: Paul Donald <[email protected]> Co-authored-by: Sébastien WENSKE <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
syncthing/syncthing#10045