Skip to content

Conversation

@jiasli
Copy link
Member

@jiasli jiasli commented Nov 7, 2024

Fix #4734

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 799de76:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/azure-cli-sp-tutorial-3.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 22f9b59:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/authenticate-azure-cli-service-principal.md ✅Succeeded View
docs-ref-conceptual/azure-cli-sp-tutorial-3.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 9987813:

⚠️ Validation status: warnings

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@jiasli jiasli reopened this Nov 8, 2024
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 799de76:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/azure-cli-sp-tutorial-3.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 799de76:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/azure-cli-sp-tutorial-3.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@jiasli jiasli changed the title Update document for using --cert Update document for using --cert in az ad sp create-for-rbac Nov 8, 2024
@jiasli jiasli marked this pull request as ready for review November 8, 2024 11:17
Copy link
Collaborator

@dbradish-microsoft dbradish-microsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jiasli , the "create new" and "use existing" H2 sections have conflicting instructions as to the format of the PEM file.

image

@jiasli jiasli force-pushed the main branch 2 times, most recently from 4bf7e9a to 6e7ceee Compare November 14, 2024 01:22
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 4bf7e9a:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/azure-cli-sp-tutorial-3.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 6e7ceee:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/azure-cli-sp-tutorial-3.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 46862d7:

✅ Validation status: passed

File Status Preview URL Details
docs-ref-conceptual/azure-cli-sp-tutorial-3.md ✅Succeeded View
docs-ref-conceptual/media/sp-tutorial/pem-file.png ✅Succeeded n/a (file deleted or renamed)

For more details, please refer to the build report.

For any questions, please:

@jiasli jiasli merged commit 288522f into MicrosoftDocs:main Nov 14, 2024
@jiasli jiasli deleted the main branch November 14, 2024 02:04
@jikuja
Copy link

jikuja commented Nov 14, 2024

@jiasli , the "create new" and "use existing" H2 sections have conflicting instructions as to the format of the PEM file.

Not really:

  • Create new creates aa new secret key and certificate and they are saved into single file
    • this format is required when client logs in
  • Use existing(=setup keyCredential on Entra ID application) requires only certificate
    • this format is recommended when setting up entra id side

Current tooling has some un-optimized workflows that cause issues like this.

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.

Article for service creation principal has misleading guidance for --cert

4 participants