Skip to content

Commit 6b7ed08

Browse files
Merge pull request #513 from microsoft/psl-ficPrincipalTypeIssue
fix: add principalType as 'ServicePrincipal' for role assignments
2 parents b1c949b + 71fc03c commit 6b7ed08

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

infra/deploy_app_service.bicep

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -345,6 +345,7 @@ resource searchIndexDataReaderAssignment 'Microsoft.Authorization/roleAssignment
345345
properties: {
346346
roleDefinitionId: searchIndexDataReader.id
347347
principalId: Website.identity.principalId
348+
principalType: 'ServicePrincipal'
348349
}
349350
}
350351

infra/main.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2291,7 +2291,8 @@
22912291
"name": "[guid(parameters('WebsiteName'), parameters('aiSearchName'), resourceId('Microsoft.Authorization/roleDefinitions', '1407120a-92aa-4202-b7e9-c0e197c71c8f'))]",
22922292
"properties": {
22932293
"roleDefinitionId": "[resourceId('Microsoft.Authorization/roleDefinitions', '1407120a-92aa-4202-b7e9-c0e197c71c8f')]",
2294-
"principalId": "[reference(resourceId('Microsoft.Web/sites', parameters('WebsiteName')), '2020-06-01', 'full').identity.principalId]"
2294+
"principalId": "[reference(resourceId('Microsoft.Web/sites', parameters('WebsiteName')), '2020-06-01', 'full').identity.principalId]",
2295+
"principalType": "ServicePrincipal"
22952296
},
22962297
"dependsOn": [
22972298
"[resourceId('Microsoft.Web/sites', parameters('WebsiteName'))]"

0 commit comments

Comments
 (0)