sql-docs icon indicating copy to clipboard operation
sql-docs copied to clipboard

Update authentication-aad-configure.md

Open JFolberth opened this issue 1 year ago • 6 comments

This pull request includes a small change to the azure-sql/database/authentication-aad-configure.md file. The change advises associating the Microsoft Entra admin ID with an Entra Group for better management.

  • azure-sql/database/authentication-aad-configure.md: Added a recommendation to tie the Microsoft Entra admin ID to an Entra Group, specifying that the Client ID of the Entra Group will be inserted into the sys.database_principals in the master table.

JFolberth avatar Aug 12 '24 20:08 JFolberth

@JFolberth : Thanks for your contribution! The author(s) have been notified to review your proposed change.

prmerger-automator[bot] avatar Aug 12 '24 20:08 prmerger-automator[bot]

Learn Build status updates of commit ed9ee5a:

:white_check_mark: Validation status: passed

File Status Preview URL Details
azure-sql/database/authentication-aad-configure.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

@nofield Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

Jak-MS avatar Aug 12 '24 21:08 Jak-MS

Learn Build status updates of commit 4928d04:

:white_check_mark: Validation status: passed

File Status Preview URL Details
azure-sql/database/authentication-aad-configure.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

@JFolberth Thanks for taking the time to submit this PR. This may be good guidance to make here, running it past some engineers to confirm if this isn't a little more complicated. Will update the PR accordingly.

WilliamDAssafMSFT avatar Sep 14 '24 01:09 WilliamDAssafMSFT

Thanks @JFolberth for your suggestion here. This and other feedback has been added to the Entra docs, please see here: https://learn.microsoft.com/en-us/azure/azure-sql/database/authentication-aad-overview?view=azuresql#microsoft-entra-administrator

nofield avatar Sep 30 '24 17:09 nofield

Marking this as resolved.

#please-close

rwestMSFT avatar Oct 15 '24 06:10 rwestMSFT