Closes #3744 and #3753#3745
Conversation
|
@reshmee011 - should we update the cmdlet permissions as well for this ? Also, I think we need to mention it in the docs as well. |
|
@gautamdsheth , it's quite interesting as I could not find any documentation on these two extra permissions from the o365 management APIs and they can't be added to the app registration from the UI except updating the manifest json to include these extra permissions. |
|
@reshmee011 - I meant in the cmdlet , we have requiredminimalpermissions attribute, maybe we can add it there ? In the cmdlet docs as well something similar maybe ? |
|
@gautamdsheth : I have added the permissions to the cmd and doc. At least adding these extra permissions helps with the unauthorized bug I was facing, hope it will be the same for others. |
|
Awesome, thanks @reshmee011 merged it !! |
Type
Related Issues?
Fixes #3744
What is in this Pull Request ?
updated permissions in file src\Commands\Resources\O365ManagementPermissions.json to include these two
{
"adminConsentDescription": "Allows the application to read service health information for your organization.",
"adminConsentDisplayName": "Read activity reports for your organization",
"id": "b3b78c39-cb1d-4d17-820a-25d9196a800e",
"isEnabled": true,
"isAdmin": true,
"consentDescription": "Allows the application to read service health information for your organization.",
"consentDisplayName": "Read service health information for your organization",
"value": "ActivityReports.Read"
},
{
"adminConsentDescription": "Allows the application to read threat intelligence data for your organization",
"adminConsentDisplayName": "Read threat intelligence data for your organization",
"id": "17f1c501-83cd-414c-9064-cd10f7aef836",
"isEnabled": true,
"isAdmin": true,
"consentDescription": "Allows the application to read threat intelligence data for your organization",
"consentDisplayName": "Read threat intelligence data for your organization",
"value": "ThreatIntelligence.Read"
}
Guidance