Skip to content

Add authentication policy configuration to the config catalog#11477

Draft
manodyaSenevirathne wants to merge 1 commit into
wso2:masterfrom
manodyaSenevirathne:master
Draft

Add authentication policy configuration to the config catalog#11477
manodyaSenevirathne wants to merge 1 commit into
wso2:masterfrom
manodyaSenevirathne:master

Conversation

@manodyaSenevirathne

Copy link
Copy Markdown
Contributor

@coderabbitai

coderabbitai Bot commented Jun 9, 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: 5caebfd5-18eb-40e4-a8de-690c47289a79

📥 Commits

Reviewing files that changed from the base of the PR and between 9d8653d and 25169f7.

📒 Files selected for processing (3)
  • en/docs/reference/config-catalog.md
  • en/tools/config-catalog-generator/data/authentication_policy.toml
  • en/tools/config-catalog-generator/data/configs.json

📝 Walkthrough

Overview

This pull request adds authentication policy configuration support to the documentation and configuration catalog. It includes the necessary configuration definitions and documentation updates to enable users to configure the authentication policy settings.

Changes

Files Modified: 3

  • en/docs/reference/config-catalog.md - Added documentation for the Authentication Policy configuration section, detailing the disable_account_disable_handler setting including its type (boolean), default value, possible values, and description.

  • en/tools/config-catalog-generator/data/authentication_policy.toml - Added new configuration file for the Authentication Policy section with the disable_account_disable_handler setting initialized to false.

  • en/tools/config-catalog-generator/data/configs.json - Added new "Authentication Policy" entry to the configuration catalog registry, defining the disable_account_disable_handler parameter and referencing the authentication policy configuration file.

Impact

The changes enable the config catalog generator to recognize and document the Authentication Policy configuration options, making them available in the generated documentation for API Management users.

Walkthrough

This PR introduces documentation for a new Authentication Policy configuration section. The change spans three files: a metadata definition in configs.json that registers the Authentication Policy option with its disable_account_disable_handler boolean parameter, a TOML example file demonstrating the configuration, and a rendered documentation section in the config catalog that displays the parameter details including type, default value, and description.

Suggested reviewers

  • chamilaadhi
  • tharikaGitHub
  • ashera96
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is incomplete. It only provides a purpose statement referencing a ported PR but omits required sections like Goals, Approach, User stories, Release notes, and others specified in the template. Complete the PR description by filling in the remaining required sections from the template, particularly Goals, Approach, Release notes, and Documentation sections.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding authentication policy configuration to the config catalog, which is reflected in the file modifications.
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 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 and usage tips.

@manodyaSenevirathne manodyaSenevirathne marked this pull request as draft June 22, 2026 04:40
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