Skip to content

feat: hydra token prefixes#2569

Open
alnr wants to merge 1 commit into
masterfrom
alnr/token-prefix
Open

feat: hydra token prefixes#2569
alnr wants to merge 1 commit into
masterfrom
alnr/token-prefix

Conversation

@alnr
Copy link
Copy Markdown
Contributor

@alnr alnr commented May 20, 2026

Ref https://github.com/ory-corp/cloud/issues/11951
Ref https://github.com/ory-corp/cloud/pull/11974

Summary by CodeRabbit

  • Documentation
    • Added new documentation entry for OAuth2 token prefix configuration under self-hosted deployment guides, now accessible in the Delegated Authentication & Federated Authorization section.

Review Change Stack

@alnr alnr self-assigned this May 20, 2026
@alnr alnr requested a review from vinckr as a code owner May 20, 2026 15:33
Copilot AI review requested due to automatic review settings May 20, 2026 15:33
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: b5768527-1ccc-4979-8a5e-7ec309316b55

📥 Commits

Reviewing files that changed from the base of the PR and between 743cca6 and 96a92ea.

⛔ Files ignored due to path filters (1)
  • docs/self-hosted/oel/oauth2/token-prefix.mdx is excluded by !**/*.mdx
📒 Files selected for processing (1)
  • sidebars-network.ts

📝 Walkthrough

Walkthrough

This pull request adds a single documentation entry reference to the network sidebar configuration. The self-hosted/oel/oauth2/token-prefix document is now included in the Token management section under Delegated AuthZ & Federated AuthN.

Changes

Documentation Navigation Update

Layer / File(s) Summary
Token prefix documentation link
sidebars-network.ts
Added self-hosted/oel/oauth2/token-prefix document entry to the "Token management" items list in the sidebar navigation structure.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A sidebar grows, one entry more,
Token prefix through the documentation door,
Navigation paths now guide the way,
OAuth flows explained with every page! 🗺️

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description provides only two external references without addressing the template sections like big picture explanation, related issue context, or checklist items. Complete the description by explaining the feature's purpose, linking related issues/documents, and confirming checklist items have been addressed.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat: hydra token prefixes' is directly related to the change, which adds documentation for token prefix functionality.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch alnr/token-prefix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation and navigation for configuring OAuth2 token prefixes in Ory Hydra, covering both Ory Network Enterprise and Ory Enterprise License (self-hosted) usage.

Changes:

  • Exposes the existing token prefix doc in the Ory Network (Hydra) sidebar under “Token management”.
  • Updates the token prefix doc to clarify Enterprise availability, document validation constraints, and add Ory Network CLI-based configuration instructions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
sidebars-network.ts Adds the token prefix doc to the Hydra → Guides → Token management section for Ory Network navigation.
docs/self-hosted/oel/oauth2/token-prefix.mdx Expands/clarifies token prefix documentation (constraints + Network/OEL configuration instructions + examples).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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