Skip to content

docs: Document Ephemeral Sessions limitation#1167

Merged
NandanPrabhu merged 1 commit into
mainfrom
docs/online-access-ephemeral-sessions
Jul 20, 2026
Merged

docs: Document Ephemeral Sessions limitation#1167
NandanPrabhu merged 1 commit into
mainfrom
docs/online-access-ephemeral-sessions

Conversation

@NandanPrabhu

@NandanPrabhu NandanPrabhu commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes the incorrect reference to a tenant feature flag for Online Access — this feature is Early Access, not gated by a flag, and is already documented as such via the existing Early Access note.
  • Documents that Online Refresh Tokens do not currently support resource servers with Ephemeral Sessions enabled, and what happens if both are enabled together (invalid_grant on the next refresh).

Test plan

  • Docs-only change; no code affected

@NandanPrabhu
NandanPrabhu requested a review from a team as a code owner July 20, 2026 09:11
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The example documentation replaces the tenant-level online refresh token prerequisite with a resource-server allow_online_access requirement and warns that Ephemeral Sessions must be disabled for online refresh tokens.

Changes

Online refresh token documentation

Layer / File(s) Summary
Resource server requirements
EXAMPLES.md
Online refresh token guidance now requires allow_online_access on the resource server and warns that Ephemeral Sessions must be disabled.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the new documentation about the Ephemeral Sessions limitation for Online Refresh Tokens.
✨ 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 docs/online-access-ephemeral-sessions

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@EXAMPLES.md`:
- Around line 850-851: Update the blockquote formatting around the [!WARNING]
callout in EXAMPLES.md so the separating line is an actual blank line rather
than a blank quoted line, resolving markdownlint MD028 while keeping the warning
as a separate callout.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 621763ad-58b0-4c46-bf2e-cd9c414060a6

📥 Commits

Reviewing files that changed from the base of the PR and between 75c5ef2 and 659699e.

📒 Files selected for processing (1)
  • EXAMPLES.md

Comment thread EXAMPLES.md
…essions limitation

Online Refresh Tokens are Early Access, not gated by a tenant feature
flag, so drop the incorrect reference. Also document that Online
Refresh Tokens do not yet support resource servers with Ephemeral
Sessions enabled.
@NandanPrabhu
NandanPrabhu force-pushed the docs/online-access-ephemeral-sessions branch from 0f57d61 to 081ac72 Compare July 20, 2026 09:26
@NandanPrabhu NandanPrabhu changed the title docs: remove online_refresh_tokens flag mention, document Ephemeral Sessions limitation docs: Document Ephemeral Sessions limitation Jul 20, 2026
@NandanPrabhu
NandanPrabhu merged commit af6d70c into main Jul 20, 2026
16 of 19 checks passed
@NandanPrabhu
NandanPrabhu deleted the docs/online-access-ephemeral-sessions branch July 20, 2026 10:50
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