Skip to content

[7.3] Feature: Graceful refresh token#6222

Merged
Thumimku merged 1 commit into
wso2:masterfrom
Thumimku:graceful-7.3
Jun 30, 2026
Merged

[7.3] Feature: Graceful refresh token#6222
Thumimku merged 1 commit into
wso2:masterfrom
Thumimku:graceful-7.3

Conversation

@Thumimku

Copy link
Copy Markdown
Contributor

@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 95c45364-3295-4398-b913-dfd6d5a0f078

📥 Commits

Reviewing files that changed from the base of the PR and between 0497bc7 and 82629a6.

📒 Files selected for processing (2)
  • en/includes/guides/fragments/manage-app/oidc-settings/refresh-token.md
  • en/includes/references/tokens/refresh-tokens.md

📝 Walkthrough

Walkthrough

Two documentation template conditionals are updated: the is_version threshold controlling refresh token expiry and rotation content for WSO2 Identity Server changes from > "7.3.0" to > "7.2.0".

Changes

Refresh Token Version Threshold

Layer / File(s) Summary
Version threshold correction in refresh token docs
en/includes/guides/fragments/manage-app/oidc-settings/refresh-token.md, en/includes/references/tokens/refresh-tokens.md
Both Jinja conditionals gating refresh token expiry and rotation content now apply to IS versions greater than 7.2.0 instead of 7.3.0.

Possibly related PRs

  • wso2/docs-is#6173: Adds graceful refresh token rotation logic to the same OIDC refresh token guide and refresh tokens reference docs that this PR modifies.

Suggested labels

Team/API Access Mgt & Authorization

Suggested reviewers

  • ashanthamara
  • himeshsiriwardana
  • hwupathum
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description only fills Purpose and omits the required Related PRs, Test environment, and Security checks sections. Add the missing template sections with related PRs, full test environment details, and completed security check items.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and matches the main change: a graceful refresh token backport affecting refresh token docs.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@Thumimku Thumimku merged commit 8a71639 into wso2:master Jun 30, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants