{"action":"created","issue":{"url":"https://api.github.com/repos/intridea/oauth2/issues/281","repository_url":"https://api.github.com/repos/intridea/oauth2","labels_url":"https://api.github.com/repos/intridea/oauth2/issues/281/labels{/name}","comments_url":"https://api.github.com/repos/intridea/oauth2/issues/281/comments","events_url":"https://api.github.com/repos/intridea/oauth2/issues/281/events","html_url":"https://github.com/intridea/oauth2/pull/281","id":192753035,"number":281,"title":"Handle redirect_uri according to OAuth 2 spec","user":{"login":"bjeanes","id":2560,"avatar_url":"https://avatars.githubusercontent.com/u/2560?v=3","gravatar_id":"","url":"https://api.github.com/users/bjeanes","html_url":"https://github.com/bjeanes","followers_url":"https://api.github.com/users/bjeanes/followers","following_url":"https://api.github.com/users/bjeanes/following{/other_user}","gists_url":"https://api.github.com/users/bjeanes/gists{/gist_id}","starred_url":"https://api.github.com/users/bjeanes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bjeanes/subscriptions","organizations_url":"https://api.github.com/users/bjeanes/orgs","repos_url":"https://api.github.com/users/bjeanes/repos","events_url":"https://api.github.com/users/bjeanes/events{/privacy}","received_events_url":"https://api.github.com/users/bjeanes/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-12-01T04:39:57Z","updated_at":"2016-12-01T05:12:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/intridea/oauth2/pulls/281","html_url":"https://github.com/intridea/oauth2/pull/281","diff_url":"https://github.com/intridea/oauth2/pull/281.diff","patch_url":"https://github.com/intridea/oauth2/pull/281.patch"},"body":"Passing redirect_uri to authorization server at grant time is optional (though encouraged and very commonly enforced by providers). However, if it is passed (as it often is) it is REQUIRED to be passed when using the grant code to get an access token.\r\n\r\nThis change allows a user to optionally provide a `redirect_uri` when configuring the client to have it passed as a parameter in both the redirect to the authorization server and the later token exchange.\r\n\r\nRelevant sections of the spec:\r\n\r\n+ https://tools.ietf.org/html/rfc6749#section-4.1\r\n+ https://tools.ietf.org/html/rfc6749#section-4.1.3\r\n+ https://tools.ietf.org/html/rfc6749#section-4.2.1\r\n+ https://tools.ietf.org/html/rfc6749#section-10.6\r\n\r\nThis change is backwards compatible, its use is optional, and it handles more of the spec so that the user does not have to.\r\n\r\nNote: this will conflict with #280. If both are cleared for merging, I'll be happy to rebase the other for a clean merge promptly."},"comment":{"url":"https://api.github.com/repos/intridea/oauth2/issues/comments/264080731","html_url":"https://github.com/intridea/oauth2/pull/281#issuecomment-264080731","issue_url":"https://api.github.com/repos/intridea/oauth2/issues/281","id":264080731,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars.githubusercontent.com/u/2354108?v=3","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2016-12-01T05:12:47Z","updated_at":"2016-12-01T05:12:47Z","body":"\n[![Coverage Status](https://coveralls.io/builds/9075734/badge)](https://coveralls.io/builds/9075734)\n\nCoverage increased (+0.07%) to 95.611% when pulling **eb2e317a633216325d2e886500e6676f407021c0 on covidence:send-redirect_uri-for-token-exchange** into **c9bbb9c58ded9b330ee105778f66f86806335d63 on intridea:master**.\n"}}
{"actor":{"display_login":"coveralls"}}