Skip to content

doc: add auth_secret_file for smtp in document#5036

Merged
SoloJacobs merged 1 commit intoprometheus:mainfrom
nutmos:feat/auth_secret-doc
Feb 22, 2026
Merged

doc: add auth_secret_file for smtp in document#5036
SoloJacobs merged 1 commit intoprometheus:mainfrom
nutmos:feat/auth_secret-doc

Conversation

@nutmos
Copy link
Contributor

@nutmos nutmos commented Feb 22, 2026

Pull Request Checklist

Please check all the applicable boxes.

  • Please list all open issue(s) discussed with maintainers related to this change
    • Fixes #
  • Is this a new Receiver integration?
  • Is this a bugfix?
    • I have added tests that can reproduce the bug which pass with this bugfix applied
  • Is this a new feature?
    • I have added tests that test the new feature's functionality
  • Does this change affect performance?
    • I have provided benchmarks comparison that shows performance is improved or is not degraded
      • You can use benchstat to compare benchmarks
    • I have added new benchmarks if required or requested by maintainers
  • Is this a breaking change?
    • My changes do not break the existing cluster messages
    • My changes do not break the existing api
  • I have added/updated the required documentation
  • I have signed-off my commits
  • I will follow best practices for contributing to this project

Which user-facing changes does this PR introduce?

Add smtp_auth_secret_file in global config and auth_secret_file in email config

PR #3087 added smtp_auth_secret_file in global config and auth_secret_file in email config but the document hasn't been updated yet. This PR updates the document for these parameters.

Signed-off-by: Nattapong Ekudomsuk <nuttapong_mos@hotmail.com>
Copy link
Contributor

@SoloJacobs SoloJacobs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@SoloJacobs SoloJacobs merged commit 62d4c4d into prometheus:main Feb 22, 2026
11 of 12 checks passed
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