Skip to content

Update Azure authentication instructions to include platform type during setup#745

Merged
tamalchowdhury merged 3 commits into
kinde-oss:mainfrom
ahn-nath:patch-5
Jun 11, 2026
Merged

Update Azure authentication instructions to include platform type during setup#745
tamalchowdhury merged 3 commits into
kinde-oss:mainfrom
ahn-nath:patch-5

Conversation

@ahn-nath

@ahn-nath ahn-nath commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Description

Updates Step 3: Add the callback URL to your Entra ID app in the Microsoft Entra ID enterprise connection doc to clarify that the redirect URI platform type in Entra must be Web, not SPA, even when the customer’s application is a single-page app.

Microsoft redirects to Kinde’s server to complete authentication, not directly to the browser app. SPA platform settings use different OAuth rules and cause login failures in this flow. This change addresses customer feedback from a support thread, where a customer reported confusion because Web worked but SPA did not.

Related issues & labels

  • Suggested label: documentation

Summary by CodeRabbit

  • Documentation
    • Clarified Step 2/Step 3 flow in the Azure enterprise connection guide for a smoother setup.
    • Added explicit instruction to set the redirect URI platform to Web (not SPA) for correct Entra ID redirects.
    • Moved "add any upstream params" before callback URL configuration and expanded step-by-step callback/redirect-URI instructions.

Clarified steps for adding the callback URL and specified platform type for redirect URI in Entra ID app configuration.
@ahn-nath ahn-nath requested a review from a team as a code owner June 3, 2026 19:45
@coderabbitai

coderabbitai Bot commented Jun 3, 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: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b5c27e5d-2631-4e75-a1da-5da4749e16b5

📥 Commits

Reviewing files that changed from the base of the PR and between b376f1a and d0fbd9e.

📒 Files selected for processing (1)
  • src/content/docs/authenticate/enterprise-connections/azure.mdx
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/content/docs/authenticate/enterprise-connections/azure.mdx

Walkthrough

Updated Azure Entra ID enterprise connection docs: moved "Add any upstream params" before callback URL instructions, expanded Step 3 into a numbered procedure that adds the callback URL, requires the redirect-URI platform be set to Web (not SPA), then Save.

Changes

Azure Enterprise Connection Callback Setup Guide

Layer / File(s) Summary
Callback URL setup documentation and step flow
src/content/docs/authenticate/enterprise-connections/azure.mdx
Rewrote Step 3 into a numbered procedure: select Redirect URIs, add the callback URL, explicitly set redirect URI platform to Web (not SPA), then select Save; moved "Add any upstream params" before the callback URL section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A little change to guide the way,
Steps aligned so none will stray,
Web not SPA, the note is clear,
Azure callbacks now appear,
Hops of joy for docs today!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately captures the main change: clarifying that the redirect URI platform type must be set to Web during Azure/Entra ID setup.
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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/content/docs/authenticate/enterprise-connections/azure.mdx`:
- Line 130: The list item contains a truncated link "Add any [upstream
params](/authenticate/auth-guides/p" which breaks MDX; replace it with the
complete link and text (e.g., "Add any [upstream
params](/authenticate/auth-guides/upstream-params)"), ensuring the closing
bracket and parenthesis are present and the surrounding sentence is finished so
the line reads as a full list item referencing the upstream params doc.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 4965c9b3-402d-442c-a367-0d9cc0d0d4e0

📥 Commits

Reviewing files that changed from the base of the PR and between 3ded71b and b376f1a.

📒 Files selected for processing (1)
  • src/content/docs/authenticate/enterprise-connections/azure.mdx

Comment thread src/content/docs/authenticate/enterprise-connections/azure.mdx Outdated
@tamalchowdhury

Copy link
Copy Markdown
Collaborator

Taking a look

@tamalchowdhury tamalchowdhury left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks all good to me.

@tamalchowdhury tamalchowdhury merged commit af3705d into kinde-oss:main Jun 11, 2026
5 checks passed
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