-
Notifications
You must be signed in to change notification settings - Fork 835
Update overview-assign-app-owners.md #1842
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Updating per internal Epic 1436967
|
Learn Build status updates of commit 1b38a4a: ✅ Validation status: passed
For more details, please refer to the build report. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds a new NOTE block to the enterprise application ownership documentation to clarify limitations for application owners added through non-UI methods (Graph API, PowerShell). This addresses internal Epic 1436967 by documenting that such owners have restricted management capabilities compared to those added through the Entra admin center.
Key Changes
- Added a NOTE block explaining that application owners added via Graph API or PowerShell have limited access to certain enterprise application settings
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| > The application may have more permissions than the owner, and thus would be an elevation of privilege over what the owner has access to as a user. An application owner can create or update users or other objects while impersonating the application. The elevation of privilege to owners can raise a security concern in some cases depending on the application's permissions. | ||
| > [!NOTE] | ||
| > Currently due to background applications and service principals objects settings dependencies, the application owners added by other than Entra admin center means (Graph API, PowerShell), will not be able to manage some enterprise applications settings like attributes and claims, modify configured SAML certificates properties or token encryption settings, etc. |
Copilot
AI
Dec 5, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sentence is overly long and complex, making it difficult to read. Consider breaking it into two sentences or restructuring for better clarity. For example: "Currently, due to background applications and service principals object settings dependencies, application owners added by means other than the Entra admin center (such as Graph API or PowerShell) have limited management capabilities. They will not be able to manage certain enterprise application settings, such as attributes and claims, SAML certificate properties, or token encryption settings."
| > Currently due to background applications and service principals objects settings dependencies, the application owners added by other than Entra admin center means (Graph API, PowerShell), will not be able to manage some enterprise applications settings like attributes and claims, modify configured SAML certificates properties or token encryption settings, etc. | |
| > Currently, due to dependencies related to background applications and service principal object settings, application owners who are added by means other than the Entra admin center (such as Graph API or PowerShell) have limited management capabilities. They will not be able to manage certain enterprise application settings, such as attributes and claims, SAML certificate properties, or token encryption settings. |
|
@secherka : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
@secherka - This PR was opened in the public repo. FTEs should work in the private repo if possible. I see that you submitted a PR to the private repo last year. If feasible, add a Otherwise, @omondiatieno - Could you please review and sign off if you approve? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
Updating per internal Epic 1436967 https://identitydivision.visualstudio.com/Engineering/_workitems/edit/1436967