{"action":"created","issue":{"url":"https://api.github.com/repos/grails/grails-core/issues/10025","repository_url":"https://api.github.com/repos/grails/grails-core","labels_url":"https://api.github.com/repos/grails/grails-core/issues/10025/labels{/name}","comments_url":"https://api.github.com/repos/grails/grails-core/issues/10025/comments","events_url":"https://api.github.com/repos/grails/grails-core/issues/10025/events","html_url":"https://github.com/grails/grails-core/issues/10025","id":163558221,"number":10025,"title":"grails 3.0.17 - Double decoding","user":{"login":"shiranr","id":1862328,"avatar_url":"https://avatars.githubusercontent.com/u/1862328?v=3","gravatar_id":"","url":"https://api.github.com/users/shiranr","html_url":"https://github.com/shiranr","followers_url":"https://api.github.com/users/shiranr/followers","following_url":"https://api.github.com/users/shiranr/following{/other_user}","gists_url":"https://api.github.com/users/shiranr/gists{/gist_id}","starred_url":"https://api.github.com/users/shiranr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shiranr/subscriptions","organizations_url":"https://api.github.com/users/shiranr/orgs","repos_url":"https://api.github.com/users/shiranr/repos","events_url":"https://api.github.com/users/shiranr/events{/privacy}","received_events_url":"https://api.github.com/users/shiranr/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/grails/grails-core/labels/Blocker","name":"Blocker","color":"e11d21"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/grails/grails-core/milestones/171","html_url":"https://github.com/grails/grails-core/milestones/grails-3.1.10","labels_url":"https://api.github.com/repos/grails/grails-core/milestones/171/labels","id":1854401,"number":171,"title":"grails-3.1.10","description":"","creator":{"login":"graemerocher","id":66626,"avatar_url":"https://avatars.githubusercontent.com/u/66626?v=3","gravatar_id":"","url":"https://api.github.com/users/graemerocher","html_url":"https://github.com/graemerocher","followers_url":"https://api.github.com/users/graemerocher/followers","following_url":"https://api.github.com/users/graemerocher/following{/other_user}","gists_url":"https://api.github.com/users/graemerocher/gists{/gist_id}","starred_url":"https://api.github.com/users/graemerocher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/graemerocher/subscriptions","organizations_url":"https://api.github.com/users/graemerocher/orgs","repos_url":"https://api.github.com/users/graemerocher/repos","events_url":"https://api.github.com/users/graemerocher/events{/privacy}","received_events_url":"https://api.github.com/users/graemerocher/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":0,"state":"open","created_at":"2016-06-29T06:23:19Z","updated_at":"2016-07-04T07:02:31Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2016-07-03T11:27:19Z","updated_at":"2016-07-06T08:02:14Z","closed_at":null,"body":"Hi,\r\n\r\nIt seems there is double decoding for grails 3.0.17 when a parameter is being transferred as a path param.\r\nFor reproduction see the following project:\r\nhttps://github.com/shiranr/bug\r\n1. Run the project\r\n2. Go to the URL: http://localhost:8080/1%2B2/view\r\n3. In the interceptor log, you will find the line: URI: /1%2B2/view BUT PARAM IS: 1 2\r\nFor some reason, the %2B is being double decoded to space instead of staying as '+'.\r\n\r\nThanks,\r\nShiran"},"comment":{"url":"https://api.github.com/repos/grails/grails-core/issues/comments/230704645","html_url":"https://github.com/grails/grails-core/issues/10025#issuecomment-230704645","issue_url":"https://api.github.com/repos/grails/grails-core/issues/10025","id":230704645,"user":{"login":"liorhson","id":6830400,"avatar_url":"https://avatars.githubusercontent.com/u/6830400?v=3","gravatar_id":"","url":"https://api.github.com/users/liorhson","html_url":"https://github.com/liorhson","followers_url":"https://api.github.com/users/liorhson/followers","following_url":"https://api.github.com/users/liorhson/following{/other_user}","gists_url":"https://api.github.com/users/liorhson/gists{/gist_id}","starred_url":"https://api.github.com/users/liorhson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/liorhson/subscriptions","organizations_url":"https://api.github.com/users/liorhson/orgs","repos_url":"https://api.github.com/users/liorhson/repos","events_url":"https://api.github.com/users/liorhson/events{/privacy}","received_events_url":"https://api.github.com/users/liorhson/received_events","type":"User","site_admin":false},"created_at":"2016-07-06T08:02:14Z","updated_at":"2016-07-06T08:02:14Z","body":"Hey @graemerocher,\r\n\r\nBecause this issue is indeed blocker for production (Bintray [Grails 3.0.17]), we fork the **grails-web-url-mappings** project and removed the URLDecoder.decode()... line from 'AbstractUrlMappingInfo' class, line 95.\r\n\r\nFor the meantime, do you think this change can cause any other problems?\r\n\r\nRegards,\r\nLior\r\n"}}
{"actor":{"display_login":"liorhson"}}