Fixes handling of strings that may be encoded when passed to jclouds. Apache jclouds should not double-encode strings and should not guard against handling encoded strings. This patch makes a change ```jclouds/core``` to remove the check for encoded strings when calling ```Strings2.urlEncode()```. The changes to all of the other components are made to remove the reliance on this behavior. In jclouds blobstore, this further allows supporting strings with "%" characters -- a matter of a number of JIRA issues. You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/851 -- Commit Summary -- * Percent encoding changes. * Fix encoding handling in S3. * Azure: Handle blobs with % characters correctly. * Do not use encoded strings in STS tests. * Fix a cloudstack test that encoded parameters. * Do not use encoded strings in EC2 tests. * Do not use encoded strings in SQS API tests. * Do not use encoded strings in AWS EC2 tests. * OpenStack Nova tests: do not pass encoded strings. -- File Changes -- M apis/cloudstack/src/test/java/org/jclouds/cloudstack/features/SSHKeyPairApiTest.java (6) M apis/ec2/src/main/java/org/jclouds/ec2/compute/EC2ComputeService.java (4) M apis/ec2/src/test/java/org/jclouds/ec2/features/AMIApiExpectTest.java (4) M apis/ec2/src/test/java/org/jclouds/ec2/features/AMIApiTest.java (56) M apis/ec2/src/test/java/org/jclouds/ec2/features/EC2ElasticBlockStoreApiExpectTest.java (19) M apis/ec2/src/test/java/org/jclouds/ec2/features/ElasticBlockStoreApiTest.java (24) M apis/ec2/src/test/java/org/jclouds/ec2/features/ElasticIPAddressApiTest.java (4) M apis/ec2/src/test/java/org/jclouds/ec2/features/InstanceApiTest.java (26) M apis/ec2/src/test/java/org/jclouds/ec2/features/KeyPairApiExpectTest.java (4) M apis/ec2/src/test/java/org/jclouds/ec2/features/SecurityGroupApiExpectTest.java (4) M apis/ec2/src/test/java/org/jclouds/ec2/features/SecurityGroupApiTest.java (12) M apis/ec2/src/test/java/org/jclouds/ec2/features/SubnetApiExpectTest.java (12) M apis/ec2/src/test/java/org/jclouds/ec2/features/WindowsApiExpectTest.java (4) M apis/ec2/src/test/java/org/jclouds/ec2/features/WindowsApiTest.java (16) M apis/openstack-nova/src/test/java/org/jclouds/openstack/nova/v2_0/features/ImageApiExpectTest.java (21) M apis/openstack-nova/src/test/java/org/jclouds/openstack/nova/v2_0/features/ServerApiExpectTest.java (14) M apis/s3/src/main/java/org/jclouds/s3/binders/BindAsHostPrefixIfConfigured.java (2) M apis/s3/src/main/java/org/jclouds/s3/xml/ListBucketHandler.java (11) M apis/sqs/src/test/java/org/jclouds/sqs/features/MessageApiExpectTest.java (70) M apis/sqs/src/test/java/org/jclouds/sqs/features/PermissionApiExpectTest.java (8) M apis/sqs/src/test/java/org/jclouds/sqs/features/QueueApiExpectTest.java (28) M apis/sts/src/test/java/org/jclouds/sts/STSApiExpectTest.java (24) M blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseContainerIntegrationTest.java (17) M core/src/main/java/org/jclouds/http/Uris.java (1) M core/src/main/java/org/jclouds/rest/binders/BindAsHostPrefix.java (5) M core/src/main/java/org/jclouds/util/Strings2.java (9) M core/src/test/java/org/jclouds/util/Strings2Test.java (6) M providers/aws-ec2/src/test/java/org/jclouds/aws/ec2/features/AWSInstanceApiTest.java (2) M providers/aws-ec2/src/test/java/org/jclouds/aws/ec2/features/AWSKeyPairApiTest.java (2) M providers/azureblob/src/main/java/org/jclouds/azureblob/xml/ContainerNameEnumerationResultsHandler.java (49) -- Patch Links -- https://github.com/jclouds/jclouds/pull/851.patch https://github.com/jclouds/jclouds/pull/851.diff --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/851
