Skip to content

docs: Revert usage of unexplained <ISSUER_URL> placeholder#806

Merged
adombeck merged 1 commit intomainfrom
800-docs-fix-issuer-url
Feb 20, 2025
Merged

docs: Revert usage of unexplained <ISSUER_URL> placeholder#806
adombeck merged 1 commit intomainfrom
800-docs-fix-issuer-url

Conversation

@adombeck
Copy link
Copy Markdown
Contributor

@adombeck adombeck commented Feb 20, 2025

Commit 7872c99 replaced

https://login.microsoftonline.com/<ISSUER_ID>/v2.0

with

<ISSUER_URL>

However, ISSUER_URL is not explained in the documentation or Microsoft Entra admin center, so the user has no indication that the URL should be of the form https://login.microsoftonline.com/<ISSUER_ID>/v2.0 (where <ISSUER_ID> is the Directory (tenant) ID as mentioned earlier on the same page).

Closes #800
UDENG-6141

Commit 7872c99 replaced

   https://login.microsoftonline.com/<ISSUER_ID>/v2.0

with

   <ISSUER_URL>

However, ISSUER_URL is not explained in the documentation or Microsoft
Entra admin center, so the user has no indication that the URL should be
of the form https://login.microsoftonline.com/<ISSUER_ID>/v2.0 (where
<ISSUER_ID> is the Directory (tenant) ID as mentioned earlier on the
same page).
@adombeck adombeck requested a review from a team as a code owner February 20, 2025 15:07
@adombeck adombeck requested a review from edibotopic February 20, 2025 15:10
Copy link
Copy Markdown
Contributor

@edibotopic edibotopic left a comment

Choose a reason for hiding this comment

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

Thanks for this: LGTM.

@adombeck adombeck merged commit 65ec1d5 into main Feb 20, 2025
14 of 16 checks passed
@adombeck adombeck deleted the 800-docs-fix-issuer-url branch February 20, 2025 17:20
@adombeck adombeck changed the title Revert usage of unexplained <ISSUER_URL> placeholder docs: Revert usage of unexplained <ISSUER_URL> placeholder Feb 25, 2025
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.

doc: page configure authd

2 participants