Skip to content

Fix: Update OAuth cache timeout configuration [main]#2118

Open
wso2-engineering-bot wants to merge 1 commit intomainfrom
fixing-issue-2114-main-1774611111
Open

Fix: Update OAuth cache timeout configuration [main]#2118
wso2-engineering-bot wants to merge 1 commit intomainfrom
fixing-issue-2114-main-1774611111

Conversation

@wso2-engineering-bot
Copy link

@wso2-engineering-bot wso2-engineering-bot commented Mar 27, 2026

This PR was automatically generated by Claude AI.

Summary by CodeRabbit

  • Documentation
    • Updated OAuth cache timeout configuration guide for HTTP endpoints. Configuration moved to deployment.toml with updated TOML syntax format. Default timeout value unchanged at 3000 seconds.

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

coderabbitai bot commented Mar 27, 2026

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: ffb5f20d-b2ac-4af2-9059-9df3dd6c675d

📥 Commits

Reviewing files that changed from the base of the PR and between a811846 and 897396d.

📒 Files selected for processing (1)
  • en/docs/reference/synapse-properties/endpoint-properties.md

Walkthrough

Documentation updated to reflect OAuth cache timeout configuration for HTTP endpoints using TOML format instead of properties file syntax. Configuration location shifted from synapse.properties to deployment.toml with updated formatting conventions.

Changes

Cohort / File(s) Summary
Documentation Update
en/docs/reference/synapse-properties/endpoint-properties.md
Updated OAuth cache timeout configuration example from properties file format to TOML format under [synapse_properties] section, with quoted key syntax.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 The config hops from .properties to .toml so fine,
With brackets and quotes, the syntax now align,
3000 seconds tick, still holding the line,
Documentation blooms in configuration's design! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is minimal and incomplete compared to the repository template, missing most sections like Purpose, Goals, Approach, User stories, Release notes, etc. Expand the description to follow the template structure, including Purpose with issue links, Goals, Approach, and other required sections even if some sections are marked N/A.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: updating OAuth cache timeout configuration documentation from synapse.properties to deployment.toml.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fixing-issue-2114-main-1774611111

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.

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