feat(developer-settings): Add webhook headers field to custom integrations#117091
Draft
billyvg wants to merge 5 commits into
Draft
feat(developer-settings): Add webhook headers field to custom integrations#117091billyvg wants to merge 5 commits into
billyvg wants to merge 5 commits into
Conversation
ebd086b to
1981701
Compare
804abcd to
b76b5d1
Compare
1981701 to
77225f9
Compare
b76b5d1 to
5137251
Compare
5137251 to
7993c37
Compare
77225f9 to
2d1b8e3
Compare
2d1b8e3 to
b5a9baa
Compare
7993c37 to
15c3367
Compare
b5a9baa to
f5a6f1d
Compare
15c3367 to
16693dc
Compare
f5a6f1d to
bfda0a6
Compare
16693dc to
64757d3
Compare
457372d to
3750d98
Compare
5f27c19 to
96c4ca9
Compare
Member
Author
|
@sentry review |
3750d98 to
7e0f76c
Compare
db89ffc to
9068301
Compare
9068301 to
ce01950
Compare
7e0f76c to
4b61975
Compare
8c17023 to
9dedde2
Compare
69aa4e1 to
85cb011
Compare
85cb011 to
a847c8a
Compare
9dedde2 to
c7ab383
Compare
a847c8a to
3ad6cd7
Compare
c7ab383 to
0b1f875
Compare
0b1f875 to
e1bcb99
Compare
3ad6cd7 to
06f9f7b
Compare
e1bcb99 to
c4a985f
Compare
06f9f7b to
40c8b7a
Compare
…tions Add a Webhook Headers textarea to the Integration Details form for internal and public custom integrations, directly below Webhook URL. Each line is a 'Header-Name: value' pair sent with every outgoing webhook request. Mirrors the allowedOrigins field exactly (convert/extract multiline helpers). Masked values returned by the API round-trip safely because the backend preserves the stored value for any entry resubmitted with the mask sentinel, so the form needs no special masking logic. Supersedes #116732, which split out this UI ahead of the backend. Co-Authored-By: Claude <noreply@anthropic.com>
Hide the Webhook Headers textarea unless the owner org has the organizations:sentry-apps-custom-webhook-headers Flagpole flag. The form schema and saved value are preserved regardless, so toggling the flag does not destroy existing header configurations. Update the two tests that directly interact with the field to render with the flag enabled, and update the flag-off test for delivery. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
40c8b7a to
1fc97cf
Compare
c4a985f to
80f9cb3
Compare
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
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.
Add a Webhook Headers textarea to the Integration Details form for internal
and public custom integrations, directly below Webhook URL. Each line is a
'Header-Name: value' pair sent with every outgoing webhook request.
Mirrors the allowedOrigins field exactly (convert/extract multiline helpers).
Note that API always returns masked values for the headers. They will be updated if the values are changed. You can also add new headers while having the existing/masked header un the textarea.
Supersedes #116732, which split out this UI ahead of the backend.
Co-Authored-By: Claude noreply@anthropic.com
Stack created with GitHub Stacks CLI • Give Feedback 💬