Skip to content

Implement resource ID retrieval for Azure authentication#2918

Merged
v-regandowner merged 4 commits into
MicrosoftDocs:mainfrom
ohadbitt:patch-6
Jul 21, 2026
Merged

Implement resource ID retrieval for Azure authentication#2918
v-regandowner merged 4 commits into
MicrosoftDocs:mainfrom
ohadbitt:patch-6

Conversation

@ohadbitt

Copy link
Copy Markdown
Contributor

Added code to fetch resource ID from metadata for Azure Data Explorer authentication.

Thank you for contributing to Kusto documentation

Please add a brief comment outlining the purpose of this PR. Add links to any relevant references such as DevOps work items.

Make sure you've done the following:

  1. Acrolinx: Make sure your Acrolinx score is at least 80 (higher is better) and with 0 spelling issues.

  2. Successful build: Review the build status to make sure all files are green (Succeeded) and there are no errors, warnings, or suggestions.

  3. Preview the pages: Click each Preview URL link, scan the entire page looking for formatting issues, in particular the parts you edited.

  4. Check the Table of Contents: If you're adding a new markdown file, make sure it is linked from the table of contents.

  5. Sign off: Once the PR is finalized, add a comment with #sign-off . If you need to cancel the sign-off, add a comment with #hold-off.

    NOTE: Signing off means the document can be published at any time.

Next steps

  • All PRs to this repository are reviewed and merged by a human. Automatic merge is disabled on this repository for PRs, even with the qualifies-for-auto-merge label.
  • Once all feedback on the PR is addressed, the PR will be merged into the main branch.

Learn more about how to contribute

Added code to fetch resource ID from metadata for Azure Data Explorer authentication.
@prmerger-automator

Copy link
Copy Markdown
Contributor

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

1 similar comment
@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 93598f0:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/rest/authenticate-with-msal.md ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 6e076c1:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/rest/authenticate-with-msal.md ✅Succeeded

For more details, please refer to the build report.

@v-regandowner

Copy link
Copy Markdown
Contributor

@spelluru

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

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Jul 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the Azure Data Explorer REST authentication documentation to retrieve the correct Azure AD resource ID from the cluster’s metadata endpoint before requesting an app-only token via MSAL.

Changes:

  • Added a step to query the cluster metadata endpoint for KustoServiceResourceId.
  • Updated the MSAL AcquireTokenForClient scope construction to use the retrieved resource ID.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread data-explorer/kusto/api/rest/authenticate-with-msal.md Outdated
Updated the method for fetching resource ID from Kusto service metadata, switching from Java to C# code.
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 6340f3c:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/rest/authenticate-with-msal.md ✅Succeeded

For more details, please refer to the build report.

Comment thread data-explorer/kusto/api/rest/authenticate-with-msal.md Outdated

@yogilad yogilad left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One comment.

Updated the code sample to clarify that the MSAL authentication flow is supported for the 'ingest-' endpoint.
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 18fa484:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/rest/authenticate-with-msal.md ✅Succeeded

For more details, please refer to the build report.

@ohadbitt

Copy link
Copy Markdown
Contributor Author

#sign-off

@prmerger-automator

Copy link
Copy Markdown
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @spelluru

@spelluru

Copy link
Copy Markdown
Contributor

#sign-off

@v-regandowner
v-regandowner merged commit 2382bd8 into MicrosoftDocs:main Jul 21, 2026
3 checks passed
learn-build-service-prod Bot added a commit that referenced this pull request Jul 21, 2026
…s://github.com/MicrosoftDocs/dataexplorer-docs (branch main) (#7548)

* Clarify Parquet continuous export performance for wide tables (#2900)

* Clarify Parquet continuous export performance for wide tables

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Silas Mendes <silasmendes@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Implement resource ID retrieval for Azure authentication (#2918)

* Implement resource ID retrieval for Azure authentication

Added code to fetch resource ID from metadata for Azure Data Explorer authentication.

* Update authenticate-with-msal.md

* Refactor resource ID retrieval to C#

Updated the method for fetching resource ID from Kusto service metadata, switching from Java to C# code.

* Clarify MSAL authentication support for ingest endpoint

Updated the code sample to clarify that the MSAL authentication flow is supported for the 'ingest-' endpoint.

---------

Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: Silas Mendes <61285023+silasmendes-ms@users.noreply.github.com>
Co-authored-by: Silas Mendes <silasmendes@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service LearnBuild@microsoft.com>
Co-authored-by: ohad bitton <32278684+ohadbitt@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants