azure-sdk-for-js icon indicating copy to clipboard operation
azure-sdk-for-js copied to clipboard

[Storage] Connection string keys should be case-insensitive

Open joheredi opened this issue 4 years ago • 1 comments

Currently, the Storage SDKs parse connection strings assuming Keys are case-sensitive. However, keys are case-insensitive according to ADO.NET docs: https://docs.microsoft.com/en-us/dotnet/framework/data/adonet/connection-strings#connection-string-syntax

We should update our logic to handle keys as case-insensitive

joheredi avatar Jan 12 '21 20:01 joheredi

Nice catch. I see a .NET PR addressing the same issue: https://github.com/Azure/azure-sdk-for-net/pull/17642

We also don't deal with secondary endpoint for connect string. https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/storage/Azure.Storage.Common/src/Shared/StorageConnectionString.cs

ljian3377 avatar Jan 13 '21 05:01 ljian3377