Skip to content

Fix: Scope authorization header examples to WSO2 IS <= 7.2.0 in custom SMS provider docs#5944

Draft
Copilot wants to merge 2 commits intofixing-product-is-issue-26942-1772908411from
copilot/sub-pr-5942
Draft

Fix: Scope authorization header examples to WSO2 IS <= 7.2.0 in custom SMS provider docs#5944
Copilot wants to merge 2 commits intofixing-product-is-issue-26942-1772908411from
copilot/sub-pr-5942

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 8, 2026

Authorization header examples in the custom SMS provider "Headers" field are only relevant for WSO2 Identity Server <= 7.2.0, which lacks a dedicated Authentication field in the UI. Versions > 7.2.0 and Asgardeo expose a dedicated authentication configuration, making the auth header guidance unnecessary and potentially confusing.

Changes

  • en/includes/guides/notification-channels/configure-sms-provider.md
    • Wrapped the authorization-specific description text and examples (Basic auth, ****** multiple headers) in a Jinja2 conditional: {% if product_name == "WSO2 Identity Server" and is_version <= "7.2.0" %}
    • For all other versions, the Headers field shows a generic header example (x-api-key: your_api_key) without auth-specific guidance

Related PRs

Test environment

Security checks


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Co-authored-by: AnuradhaSK <25483865+AnuradhaSK@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix authorization header examples for custom SMS provider Fix: Scope authorization header examples to WSO2 IS <= 7.2.0 in custom SMS provider docs Mar 8, 2026
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.

3 participants