Skip to content

Commit 717ea81

Browse files
authored
Merge pull request #9 from cmbrose/id-casing-fix
Don't lowercase resourceIds
2 parents 36b36ee + 60b3b87 commit 717ea81

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

cosmos-db/cosmos-db.psm1

+8-1
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,14 @@ Function Get-Base64Masterkey([string]$ResourceGroup, [string]$Database, [string]
7171
}
7272

7373
Function Get-Signature([string]$verb, [string]$resourceType, [string]$resourceUrl, [string]$now) {
74-
((@($verb, $resourceType, $resourceUrl, $now, "") -join "`n") + "`n").ToLower()
74+
$parts = @(
75+
$verb.ToLower(),
76+
$resourceType.ToLower(),
77+
$resourceUrl,
78+
$now.ToLower(),
79+
""
80+
)
81+
(($parts -join "`n") + "`n")
7582
}
7683

7784
Function Get-Base64EncryptedSignatureHash([string]$masterKey, [string]$signature) {

tests/Get-AuthorizationHeader.Tests.ps1

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ InModuleScope cosmos-db {
3030
It "Returns the correct signature hashed with the master key" {
3131
$result = Get-AuthorizationHeader -ResourceGroup $MOCK_RG -SubscriptionId $MOCK_SUB -Database $MOCK_DB -Verb $MOCK_VERB -ResourceType $MOCK_RESOURCE_TYPE -ResourceUrl $MOCK_RESOURCE_URL -Now $MOCK_NOW
3232

33-
$expectedSignature = "$MOCK_VERB`n$MOCK_RESOURCE_TYPE`n$MOCK_RESOURCE_URL`n$MOCK_NOW`n`n".ToLower()
33+
$expectedSignature = "$($MOCK_VERB.ToLower())`n$($MOCK_RESOURCE_TYPE.ToLower())`n$MOCK_RESOURCE_URL`n$($MOCK_NOW.ToLower())`n`n"
3434

3535
$hasher = New-Object System.Security.Cryptography.HMACSHA256 -Property @{ Key = $MOCK_MASTER_KEY_BYTES }
3636
$sigBinary=[System.Text.Encoding]::UTF8.GetBytes($expectedSignature)

0 commit comments

Comments
 (0)