Skip to content

docs(github-enterprise): note github.com webhook URL for custom GitHub Apps#17808

Merged
tnt-sentry merged 1 commit into
masterfrom
tnt-sentry/feat/ghe-github-com-webhook-url
May 20, 2026
Merged

docs(github-enterprise): note github.com webhook URL for custom GitHub Apps#17808
tnt-sentry merged 1 commit into
masterfrom
tnt-sentry/feat/ghe-github-com-webhook-url

Conversation

@tnt-sentry
Copy link
Copy Markdown
Contributor

@tnt-sentry tnt-sentry commented May 19, 2026

DESCRIBE YOUR PR

Adds a small footnote under the existing Webhook URL row in the GitHub Enterprise setup table, directing customers configuring a custom GitHub App on github.com (rather than GHE Server or GHE Cloud) to the dedicated /extensions/github-enterprise/webhook/github-com/ endpoint.

Backing code change: getsentry/sentry#115599 added the dedicated route because github.com doesn't send the X-GitHub-Enterprise-Host or X-Github-Tenant headers that GHES and GHE Cloud use for host identification. The feature is now GA (getsentry/sentry-options-automator#7866).

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

@vercel
Copy link
Copy Markdown

vercel Bot commented May 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment May 19, 2026 7:25pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored May 19, 2026 7:25pm

Request Review

<td>
https://sentry.io/extensions/github-enterprise/webhook/
<br />
<small>If you're configuring a custom GitHub App on github.com (instead of GitHub Enterprise Server or GitHub Enterprise Cloud), use `https://sentry.io/extensions/github-enterprise/webhook/github-com/` instead.</small>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@tnt-sentry I'm getting a 404 on this external link.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It gives me a 400 which is expected if you navigate to it in your browser. It is meant to be used in the github app configuration. If you are getting a 400 and not 404 then its behaving properly.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah, good to know. Is it worth noting that this URL only works as a part of the integration config? I can't tell whether I should care about the functionality of those URLs from the current instructions.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

image The url is presented as part of a form of things to copy paste into the github app creation. With that context, I think it is fine as is.

Copy link
Copy Markdown
Contributor

@sfanahata sfanahata left a comment

Choose a reason for hiding this comment

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

Looks good! 🚢

@tnt-sentry tnt-sentry merged commit 21344bd into master May 20, 2026
21 checks passed
@tnt-sentry tnt-sentry deleted the tnt-sentry/feat/ghe-github-com-webhook-url branch May 20, 2026 19:30
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