{"comment":{"created_at":"2011-10-26T02:57:23Z","body":"+1","updated_at":"2011-10-26T02:57:23Z","url":"https://api.github.com/repos/seancribbs/webmachine-ruby/issues/comments/2525990","id":2525990,"user":{"avatar_url":"https://secure.gravatar.com/avatar/ce461b57b6a1f65ce5b6cc1c124673e3?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"ce461b57b6a1f65ce5b6cc1c124673e3","url":"https://api.github.com/users/seancribbs","id":1772,"login":"seancribbs"}},"action":"created","issue":{"number":15,"created_at":"2011-10-26T02:52:25Z","pull_request":{"diff_url":null,"patch_url":null,"html_url":null},"body":"Request#query currently uses URI.unescape to unquote the key/value pairs in the query string. However, URI.unescape is not as thorough as CGI.unescape; for example, consider the following:\r\n\r\n```ruby\r\nq = \"x=two+words&y=two%20words\"\r\nputs URI.unescape(q) # -> \"x=two+words&y=two words\"\r\nputs CGI.unescape(q) # -> \"x=two words&y=two words\"\r\n```\r\n\r\nIn other words, CGI.unescape knows that `+` characters should be converted to spaces.\r\n\r\nWould you have any objections to Request#query changing to use CGI.unescape instead of URI.unescape?","comments":1,"title":"URI.unescape vs. CGI.unescape","updated_at":"2011-10-26T02:57:23Z","url":"https://api.github.com/repos/seancribbs/webmachine-ruby/issues/15","id":2052686,"assignee":null,"milestone":null,"closed_at":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/992fe8c19bbbc27f2b562a9f96efc03d?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"992fe8c19bbbc27f2b562a9f96efc03d","url":"https://api.github.com/users/jamis","id":1627,"login":"jamis"},"html_url":"https://github.com/seancribbs/webmachine-ruby/issues/15","labels":[{"name":"bug","url":"https://api.github.com/repos/seancribbs/webmachine-ruby/labels/bug","color":"e10c02"}],"state":"open"},"legacy":{"issue_id":2052686,"comment_id":2525990}}