Skip to content

Fix: Improve OAuth token expiration documentation [master]#11679

Open
wso2-engineering-bot wants to merge 1 commit into
masterfrom
fixing-issue-11213-master-1784633676
Open

Fix: Improve OAuth token expiration documentation [master]#11679
wso2-engineering-bot wants to merge 1 commit into
masterfrom
fixing-issue-11213-master-1784633676

Conversation

@wso2-engineering-bot

Copy link
Copy Markdown
Contributor

This PR was automatically generated by Claude AI.

  • Issue: Documentation Improvements – OAuth Token Expiration #11213
  • Type: Broken Links / Documentation / Suggestions
  • Summary: Added clarifications that expiration times are defined in seconds and that configuration changes only apply to new applications. Fixed typo in timestamp skew configuration description.
  • Build Verification: Documentation built successfully with mkdocs build

- Add clarification that expiration times are defined in seconds
- Add note that configuration changes only apply to new applications
- Fix typo in timestamp skew configuration description

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

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: afd93972-27ee-42b8-843e-71c943805fb6

📥 Commits

Reviewing files that changed from the base of the PR and between 4b4d7e3 and daec103.

📒 Files selected for processing (2)
  • en/docs/api-developer-portal/manage-application/generate-keys/obtain-access-token/changing-the-default-token-expiration-time.md
  • en/docs/api-security/key-management/tokens/token-expiration.md

📝 Walkthrough

Summary

  • Clarified that token expiration and timestamp skew values are configured in seconds.
  • Documented that expiration configuration changes apply only to newly created applications.
  • Corrected and clarified the timestamp skew guidance.
  • Verified the documentation with mkdocs build.

Walkthrough

Updated two token expiration documentation pages. The changes clarify that configuration updates apply only to newly created applications, validity times must be specified in seconds, and timestamp skew is configured in seconds. The token expiration guidance was also rewritten to remove duplicated or malformed phrasing.

Suggested reviewers: chamilaadhi, tharikagithub

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is missing most required template sections, including Purpose, Goals, Approach, Release note, and Documentation. Add the missing template sections, especially Purpose, Goals, Approach, Release note, Documentation, and testing/security details.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly reflects the documentation update to OAuth token expiration guidance.
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
  • Commit unit tests in branch fixing-issue-11213-master-1784633676

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.

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.

1 participant