Re: [jclouds/jclouds] [JCLOUDS-1391] Sort headers alfabetically/lowercase before signing (#1185)

2018-03-07 Thread Andrew Gaul 
gaul requested changes on this pull request.

Can you add an integration test that failed previously but succeeds with this 
pull request?  Also correct spelling: alphabetically.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1185#pullrequestreview-102055650

[jclouds/jclouds] [JCLOUDS-1391] Sort headers alfabetically/lowercase before signing (#1185)

2018-03-07 Thread Alin Dreghiciu 
https://issues.apache.org/jira/browse/JCLOUDS-1391
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/1185

-- Commit Summary --

  * [JCLOUDS-1391] Sort headers alfabetically/lowercase before signing

-- File Changes --

M 
providers/azureblob/src/main/java/org/jclouds/azure/storage/filters/SharedKeyLiteAuthentication.java
 (33)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/1185.patch
https://github.com/jclouds/jclouds/pull/1185.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1185