Fixed encoding to ensure lower case % encoding

Azure / azure-cosmosdb-java

Java Async SDK for SQL API of Azure Cosmos DB

MIT License

54 stars 61 forks source link

Fixed encoding to ensure lower case % encoding #224

Closed srinathnarayanan closed 5 years ago

srinathnarayanan commented 5 years ago

URL encoded auth key will not contain charatcers like "%2B" but instead "%2b". This is needed since the backend does case sensitive decoding.

srinathnarayanan commented 5 years ago

Could you quickly add a test case that has the offending string that we struggled with before?

The offending string is the Auth header. I have added tests that perform a bunch of operations and check that the auth header being sent over the wire has the required encoding.

christopheranderson commented 5 years ago

No longer necessary