{"action":"created","issue":{"url":"https://api.github.com/repos/7digital/SevenDigital.Api.Wrapper/issues/236","repository_url":"https://api.github.com/repos/7digital/SevenDigital.Api.Wrapper","labels_url":"https://api.github.com/repos/7digital/SevenDigital.Api.Wrapper/issues/236/labels{/name}","comments_url":"https://api.github.com/repos/7digital/SevenDigital.Api.Wrapper/issues/236/comments","events_url":"https://api.github.com/repos/7digital/SevenDigital.Api.Wrapper/issues/236/events","html_url":"https://github.com/7digital/SevenDigital.Api.Wrapper/pull/236","id":139956853,"number":236,"title":"Remove mutation that was adding extra query params","user":{"login":"danhaller","id":2386801,"avatar_url":"https://avatars.githubusercontent.com/u/2386801?v=3","gravatar_id":"","url":"https://api.github.com/users/danhaller","html_url":"https://github.com/danhaller","followers_url":"https://api.github.com/users/danhaller/followers","following_url":"https://api.github.com/users/danhaller/following{/other_user}","gists_url":"https://api.github.com/users/danhaller/gists{/gist_id}","starred_url":"https://api.github.com/users/danhaller/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danhaller/subscriptions","organizations_url":"https://api.github.com/users/danhaller/orgs","repos_url":"https://api.github.com/users/danhaller/repos","events_url":"https://api.github.com/users/danhaller/events{/privacy}","received_events_url":"https://api.github.com/users/danhaller/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-03-10T17:21:53Z","updated_at":"2016-03-10T17:30:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/7digital/SevenDigital.Api.Wrapper/pulls/236","html_url":"https://github.com/7digital/SevenDigital.Api.Wrapper/pull/236","diff_url":"https://github.com/7digital/SevenDigital.Api.Wrapper/pull/236.diff","patch_url":"https://github.com/7digital/SevenDigital.Api.Wrapper/pull/236.patch"},"body":"POST endpoints with parameter substitution e.g. ~/foo/{bar} are not working after v7.  The user is told that query string parameters cannot be used in combination with payloads, even if the parameter is substituted into the url.\r\n\r\n I tracked this down to unintentional mutation in the RequestBuilder.BuildOAuthHeader method, which was adding the substitution parameters back in to the query string parameters after they had been removed.\r\n\r\nI've fixed the mutation and renamed a couple of variables to make it a little clearer."},"comment":{"url":"https://api.github.com/repos/7digital/SevenDigital.Api.Wrapper/issues/comments/194967779","html_url":"https://github.com/7digital/SevenDigital.Api.Wrapper/pull/236#issuecomment-194967779","issue_url":"https://api.github.com/repos/7digital/SevenDigital.Api.Wrapper/issues/236","id":194967779,"user":{"login":"danhaller","id":2386801,"avatar_url":"https://avatars.githubusercontent.com/u/2386801?v=3","gravatar_id":"","url":"https://api.github.com/users/danhaller","html_url":"https://github.com/danhaller","followers_url":"https://api.github.com/users/danhaller/followers","following_url":"https://api.github.com/users/danhaller/following{/other_user}","gists_url":"https://api.github.com/users/danhaller/gists{/gist_id}","starred_url":"https://api.github.com/users/danhaller/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danhaller/subscriptions","organizations_url":"https://api.github.com/users/danhaller/orgs","repos_url":"https://api.github.com/users/danhaller/repos","events_url":"https://api.github.com/users/danhaller/events{/privacy}","received_events_url":"https://api.github.com/users/danhaller/received_events","type":"User","site_admin":false},"created_at":"2016-03-10T17:30:17Z","updated_at":"2016-03-10T17:30:17Z","body":"ok, done"}}