Skip to content

fix: Identity provider authentication issue resolved - #2366

Merged
Roopan-Microsoft merged 2 commits into
Azure-Samples:devfrom
PadhiAjit-Microsoft:identity-access-fix
Aug 14, 2026
Merged

Roopan-Microsoft merged 2 commits into
Azure-Samples:devfrom
PadhiAjit-Microsoft:identity-access-fix

Conversation

@PadhiAjit-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

  • This pull request enhances the post-deployment setup scripts to ensure the deployer user is granted the "Container Apps Contributor" role at the resource group level. This direct role assignment is necessary for configuring Microsoft identity provider (Easy Auth) on container apps and avoids issues with group-based RBAC. The changes are made idempotent and include error handling for insufficient permissions.

Role assignment improvements:

  • Both post_deployment_setup.ps1 (PowerShell) and post_deployment_setup.sh (Bash) scripts now attempt to directly assign the "Container Apps Contributor" role to the deployer user at the resource group scope, ensuring manual Easy Auth configuration is possible and sidestepping group-based RBAC issues. [1] [2]
  • Added error handling and user feedback if the role assignment fails due to insufficient permissions (e.g., missing Owner or User Access Administrator role). [1] [2]

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft
Roopan-Microsoft merged commit a7c98e7 into Azure-Samples:dev Aug 14, 2026
1 of 2 checks passed
@PadhiAjit-Microsoft
PadhiAjit-Microsoft deleted the identity-access-fix branch August 14, 2026 11:36

This branch had an error being deployed

1 failed deployment
production — bec8202f Deployed Aug 14, 2026 by PadhiAjit-Microsoft via azd template validation #76
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