Skip to content

Update keyless-connections.md #472

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

Merged
merged 1 commit into from
Aug 11, 2025
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion articles/search/keyless-connections.md
Original file line number Diff line number Diff line change
Expand Up @@ -184,7 +184,7 @@ from azure.search.documents import SearchClient
from azure.identity import DefaultAzureCredential, AzureAuthorityHosts

# Azure Public Cloud
audience = "https://search.windows.net"
audience = "[https://search.windows.net](https://search.azure.com)"
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @srishtigangulyabb, I don't understand this edit. This value is the scope ("In the context of Microsoft Entra, scopes refer to the permissions or access levels granted to external identities"), and it's a string, not a URL.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

@srishtigangulyabb srishtigangulyabb Aug 11, 2025

Choose a reason for hiding this comment

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

The proposed change is from "https://search.windows.net" to "https://search.azure.com".

I have tried the code sample mentioned in the documentation but it gave me an error. When I learnt that the audience value should be "https://search.azure.com", I raised a request to update the documentation with the corrected value.

authority = AzureAuthorityHosts.AZURE_PUBLIC_CLOUD

service_endpoint = os.environ["AZURE_SEARCH_ENDPOINT"]
Expand Down