Skip to content

Add OAuth Authentication section to Webhooks Overview - #140

Draft
kavanpd wants to merge 1 commit into
mainfrom
deveco-358-oauth-webhooks-docs
Draft

Add OAuth Authentication section to Webhooks Overview#140
kavanpd wants to merge 1 commit into
mainfrom
deveco-358-oauth-webhooks-docs

Conversation

@kavanpd

@kavanpd kavanpd commented Jul 8, 2026

Copy link
Copy Markdown

Summary

  • Adds a comprehensive OAuth Authentication section to docs/webhooks/01-Overview.md covering purpose and use cases, OAuth client creation (including token validation at create/update time), associating a client with a webhook subscription via oauth_client_id, product limits (10 clients per account), and permission/scope requirements. (DEVECO-358)
  • Cross-links the new section from the Behavior page's existing OAuth Authentication blurb in docs/webhooks/02-Behavior.md, matching the pattern used by the Mutual TLS and Signature sections.

Test plan

  • Review the rendered markdown on this PR's Files tab
  • Webhooks engineering to confirm technical details (questions shared in Slack)

🤖 Generated with Claude Code

Documents the OAuth 2.0 client credentials feature for webhook
subscriptions: purpose and use cases, OAuth client creation and
token validation, subscription association, product limits, and
permission requirements. Cross-links the new section from the
Behavior page. (DEVECO-358)

Co-Authored-By: Claude <noreply@anthropic.com>

Compared to static authentication methods, OAuth provides automatic token refresh and retry behavior: if a delivery receives a `401 Unauthorized` response, PagerDuty refreshes the token and immediately retries. See [OAuth Authentication Retries](../../docs/webhooks/02-Behavior.md#oauth-authentication-retries) for complete retry behavior.

#### Creating an OAuth Client

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.

I'm not sure I see the point of adding this information here. It duplicates the authoritative source in the API reference and is just another place that can fall out of sync. I think we could link to the API reference here for the specific details about how the requests work. (Note that Claude can't read that link because it's a SPA, but may have more luck here ).


#### Permissions and Requirements

- Managing OAuth clients requires the same permissions as managing webhook subscriptions.

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.

Incorrect. It requires admin or owner permissions. In contrast, lower tier users can manage webhook subscriptions belonging to their services.

@gaberduty gaberduty left a comment

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.

I like the "purpose and use case" bit, but I think we shouldn't duplicate the actual API reference materials here, and should instead just link to them.

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