Skip to content

Commit 68da1ac

Browse files
authored
Make canonicalizedResource lower-case (#26802)
* Make canonicalizedResource lower-case
1 parent c2e25ba commit 68da1ac

10 files changed

+754
-5
lines changed

sdk/tables/Azure.Data.Tables/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
### Breaking Changes
88

99
### Bugs Fixed
10+
- Fixed an issue that caused authenticate failures when using a SAS token with a table name that contains upper-case characters. ([#26791](https://github.com/Azure/azure-sdk-for-net/issues/26791))
1011

1112
### Other Changes
1213

sdk/tables/Azure.Data.Tables/src/Sas/TableSasBuilder.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public TableSasBuilder(string tableName, string rawPermissions, DateTimeOffset e
4141
Argument.AssertNotNullOrEmpty(tableName, nameof(tableName));
4242
Argument.AssertNotNullOrEmpty(rawPermissions, nameof(tableName));
4343

44-
TableName = tableName.ToLowerInvariant();
44+
TableName = tableName;
4545
ExpiresOn = expiresOn;
4646
Permissions = rawPermissions.ToLowerInvariant();
4747
}
@@ -256,7 +256,7 @@ public string Sign(TableSharedKeyCredential sharedKeyCredential) =>
256256
/// </returns>
257257
private static string GetCanonicalName(string account, string tableName) =>
258258
// Table: "/table/account/tablename"
259-
string.Join("", new[] { "/table/", account, "/", tableName });
259+
string.Join("", new[] { "/table/", account, "/", tableName.ToLowerInvariant() });
260260

261261
/// <summary>
262262
/// Returns a string that represents the current object.

sdk/tables/Azure.Data.Tables/tests/SessionRecords/TableClientLiveTests(Storage)/ValidateSasCredentialsWithGenerateSasUriAndUpperCaseTableName.json

Lines changed: 183 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

sdk/tables/Azure.Data.Tables/tests/SessionRecords/TableClientLiveTests(Storage)/ValidateSasCredentialsWithGenerateSasUriAndUpperCaseTableNameAsync.json

Lines changed: 183 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)