-
Notifications
You must be signed in to change notification settings - Fork 316
Labels
ClientThis issue points to a problem in the data-plane of the library.This issue points to a problem in the data-plane of the library.KeyVaultKey VaultKey Vaultcustomer-reportedIssues that are reported by GitHub users external to the Azure organization.Issues that are reported by GitHub users external to the Azure organization.needs-team-attentionWorkflow: This issue needs attention from Azure service team or SDK teamWorkflow: This issue needs attention from Azure service team or SDK team
Milestone
Description
The Key Vault ResourceExt
does not canonicalize URLs, such that "https://my-vault.vault.azure.net//secrets/my-secret" fails with "missing collection". I believe other languages' SDKs handle this already. Unfortunately, it seems url::Url
does not canonicalize like .NET's System.Url
does.
FWIW, in the past when I first proposed these parsing APIs to the Key Vault v-team, I investigated and the service does indeed canonicalize URLs itself such that it treats "//" as "/".
Metadata
Metadata
Assignees
Labels
ClientThis issue points to a problem in the data-plane of the library.This issue points to a problem in the data-plane of the library.KeyVaultKey VaultKey Vaultcustomer-reportedIssues that are reported by GitHub users external to the Azure organization.Issues that are reported by GitHub users external to the Azure organization.needs-team-attentionWorkflow: This issue needs attention from Azure service team or SDK teamWorkflow: This issue needs attention from Azure service team or SDK team
Type
Projects
Status
Done
Status
Done