{"action":"created","issue":{"url":"https://api.github.com/repos/jclouds/jclouds/issues/851","labels_url":"https://api.github.com/repos/jclouds/jclouds/issues/851/labels{/name}","comments_url":"https://api.github.com/repos/jclouds/jclouds/issues/851/comments","events_url":"https://api.github.com/repos/jclouds/jclouds/issues/851/events","html_url":"https://github.com/jclouds/jclouds/pull/851","id":103593773,"number":851,"title":"JCLOUDS-217: Allow for strings with % characters in jclouds","user":{"login":"timuralp","id":703870,"avatar_url":"https://avatars.githubusercontent.com/u/703870?v=3","gravatar_id":"","url":"https://api.github.com/users/timuralp","html_url":"https://github.com/timuralp","followers_url":"https://api.github.com/users/timuralp/followers","following_url":"https://api.github.com/users/timuralp/following{/other_user}","gists_url":"https://api.github.com/users/timuralp/gists{/gist_id}","starred_url":"https://api.github.com/users/timuralp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timuralp/subscriptions","organizations_url":"https://api.github.com/users/timuralp/orgs","repos_url":"https://api.github.com/users/timuralp/repos","events_url":"https://api.github.com/users/timuralp/events{/privacy}","received_events_url":"https://api.github.com/users/timuralp/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-08-27T20:57:28Z","updated_at":"2015-08-27T21:17:18Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jclouds/jclouds/pulls/851","html_url":"https://github.com/jclouds/jclouds/pull/851","diff_url":"https://github.com/jclouds/jclouds/pull/851.diff","patch_url":"https://github.com/jclouds/jclouds/pull/851.patch"},"body":"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."},"comment":{"url":"https://api.github.com/repos/jclouds/jclouds/issues/comments/135557100","html_url":"https://github.com/jclouds/jclouds/pull/851#issuecomment-135557100","issue_url":"https://api.github.com/repos/jclouds/jclouds/issues/851","id":135557100,"user":{"login":"nacx","id":446705,"avatar_url":"https://avatars.githubusercontent.com/u/446705?v=3","gravatar_id":"","url":"https://api.github.com/users/nacx","html_url":"https://github.com/nacx","followers_url":"https://api.github.com/users/nacx/followers","following_url":"https://api.github.com/users/nacx/following{/other_user}","gists_url":"https://api.github.com/users/nacx/gists{/gist_id}","starred_url":"https://api.github.com/users/nacx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nacx/subscriptions","organizations_url":"https://api.github.com/users/nacx/orgs","repos_url":"https://api.github.com/users/nacx/repos","events_url":"https://api.github.com/users/nacx/events{/privacy}","received_events_url":"https://api.github.com/users/nacx/received_events","type":"User","site_admin":false},"created_at":"2015-08-27T21:17:18Z","updated_at":"2015-08-27T21:17:18Z","body":"This is quite a sensible change that may break many clients.\r\n\r\nYou should also check all uses of the [SkipEncoding](https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/rest/annotations/SkipEncoding.java) annotation carefully and discuss if it is still needed. Then we need to make sure that the relevant live tests for the methods that are using it still succeed with your change."}}