Charles Lowell
Charles Lowell
No matter the contents of the output directory, the Portability Analysis Results window is always empty at startup. Cause appears to be: 1. `OptionsModel.Formats` is backed by a collection initialized...
`TokenCache.__add` pulls cacheable items from the given event and calls `modify` individually for each item. A single call to `TokenCache.add` may therefore entail up to 5 `modify` calls. This can...
Constructing `ClientApplication` entails at least one HTTP request, for tenant discovery. This adds all the costs and failure modes associated with the network to constructing the class. In my own...
I think user authentication would be more ergonomic if authentication methods returned the authenticated account. For example, when I authenticate a user via `acquire_token_by_authorization_code` I get tokens and some response...
~Blocked on #17417~ Workload Identity isn't yet supported out of the box but my comment below (https://github.com/Azure/azure-sdk-for-go/issues/15615#issuecomment-1211012677) has an implementation you can copy in the meantime for use with azidentity...
Live tests would run significantly faster if they executed test cases in parallel. That isn't possible today because our recording infrastructure isn't concurrency safe. Making it so would require at...
Key Vault and Tables have vendored `BearerTokenPolicy` and `ExpiringResource` from `azcore` to implement their challenge protocols. It would be better if they could implement just their challenge handling logic and...
Today, pollers reject relative URLs: https://github.com/Azure/azure-sdk-for-go/blob/fc95da9cfa30aa95bb62d59950267634db929195/sdk/azcore/internal/pollers/util.go#L50-L53 I don't think anything other than autorest.testserver cases is blocked by this but grepping the Python repo I see that Azure Communication Services and...
Key Vault modules contain duplicated test helper code that we could move into the `internal` module for easier maintenance.
Seems sensible to finish #14920 first and let MSAL handle this but it's uncertain when that will be possible because MSAL doesn't yet allow setting login_hint (https://github.com/AzureAD/microsoft-authentication-library-for-go/issues/271). Adding it to...