Skip to content

Conversation

@friedrichg
Copy link
Member

@friedrichg friedrichg commented Nov 6, 2025

What this PR does: This PR adds new receiver validation for discord and email to protect the disclosure of local files like other receivers.

Discovered while reviewing #7092

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg changed the title Test invalid alertmanager configs (discord and email) Add receiver validation (discord and email) Nov 7, 2025
@SungJin1212
Copy link
Member

Thanks!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 7, 2025
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg enabled auto-merge (squash) November 7, 2025 02:01
Co-authored-by: SungJin1212 <tjdwls1201@gmail.com>
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg disabled auto-merge November 7, 2025 02:06
@friedrichg friedrichg enabled auto-merge (squash) November 7, 2025 03:50
@friedrichg friedrichg merged commit 92070a4 into master Nov 7, 2025
62 of 66 checks passed
@friedrichg friedrichg deleted the add-integration-validation branch November 7, 2025 17:15
friedrichg added a commit that referenced this pull request Nov 7, 2025
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/alertmanager lgtm This PR has been approved by a maintainer size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants