Skip to content

Conversation

@0xSysR3ll
Copy link
Contributor

Description

This PR adds the ability to configure custom HTTP headers for webhook notifications.

How Has This Been Tested?

  • Added custom headers via the UI and verified they appear in webhook requests
  • Tested adding/removing multiple headers
  • Verified empty headers are automatically filtered out on save
  • Confirmed Authorization header (authHeader ) takes precedence over custom Authorization headers
  • Tested webhook test notification with custom headers (used a simple python webhook listener script to inspect headers sent in requests)

Screenshots / Logs (if applicable)

image

Checklist:

  • I have read and followed the contribution guidelines.
  • Disclosed any use of AI (see our policy)
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • Successful build pnpm build
  • Translation keys pnpm i18n:extract
  • Database migration (if required)

@0xSysR3ll 0xSysR3ll requested a review from a team as a code owner December 14, 2025 21:55
@0xSysR3ll 0xSysR3ll force-pushed the 0xsysr3ll/feat/webhook-custom-headers branch 2 times, most recently from a93935e to ef7d979 Compare December 14, 2025 22:05
@M0NsTeRRR
Copy link
Member

Confirmed Authorization header (authHeader ) takes precedence over custom Authorization headers

In my opinion, it would be better to prevent saving if the system detects that authentication headers are already configured.

@0xSysR3ll
Copy link
Contributor Author

Confirmed Authorization header (authHeader ) takes precedence over custom Authorization headers

In my opinion, it would be better to prevent saving if the system detects that authentication headers are already configured.

Mmm not a bad idea ! I'll add this :)

@fallenbagel fallenbagel changed the title 0xsysr3ll/feat/webhook custom headers feat(notifications): webhook custom headers Dec 14, 2025
Signed-off-by: 0xsysr3ll <0xsysr3ll@pm.me>
Signed-off-by: 0xsysr3ll <0xsysr3ll@pm.me>
Signed-off-by: 0xsysr3ll <0xsysr3ll@pm.me>
@0xSysR3ll 0xSysR3ll force-pushed the 0xsysr3ll/feat/webhook-custom-headers branch from 13dcaf1 to 3070ba7 Compare December 15, 2025 18:04
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.

3 participants