{"action":"created","issue":{"url":"https://api.github.com/repos/PerlDancer/Dancer2-Plugin-Auth-Extensible/issues/81","repository_url":"https://api.github.com/repos/PerlDancer/Dancer2-Plugin-Auth-Extensible","labels_url":"https://api.github.com/repos/PerlDancer/Dancer2-Plugin-Auth-Extensible/issues/81/labels{/name}","comments_url":"https://api.github.com/repos/PerlDancer/Dancer2-Plugin-Auth-Extensible/issues/81/comments","events_url":"https://api.github.com/repos/PerlDancer/Dancer2-Plugin-Auth-Extensible/issues/81/events","html_url":"https://github.com/PerlDancer/Dancer2-Plugin-Auth-Extensible/pull/81","id":283868209,"node_id":"MDExOlB1bGxSZXF1ZXN0MTU5NjQ2ODI2","number":81,"title":"change return_url back to path (instead of request_uri)","user":{"login":"M0ses","id":1991473,"node_id":"MDQ6VXNlcjE5OTE0NzM=","avatar_url":"https://avatars3.githubusercontent.com/u/1991473?v=4","gravatar_id":"","url":"https://api.github.com/users/M0ses","html_url":"https://github.com/M0ses","followers_url":"https://api.github.com/users/M0ses/followers","following_url":"https://api.github.com/users/M0ses/following{/other_user}","gists_url":"https://api.github.com/users/M0ses/gists{/gist_id}","starred_url":"https://api.github.com/users/M0ses/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/M0ses/subscriptions","organizations_url":"https://api.github.com/users/M0ses/orgs","repos_url":"https://api.github.com/users/M0ses/repos","events_url":"https://api.github.com/users/M0ses/events{/privacy}","received_events_url":"https://api.github.com/users/M0ses/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2017-12-21T12:32:49Z","updated_at":"2018-12-16T17:18:49Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/PerlDancer/Dancer2-Plugin-Auth-Extensible/pulls/81","html_url":"https://github.com/PerlDancer/Dancer2-Plugin-Auth-Extensible/pull/81","diff_url":"https://github.com/PerlDancer/Dancer2-Plugin-Auth-Extensible/pull/81.diff","patch_url":"https://github.com/PerlDancer/Dancer2-Plugin-Auth-Extensible/pull/81.patch"},"body":"Please be aware that this patch reverts commit 46cc867 and I'm not sure if this is desirable.\r\n\r\nWithout this patch, if you use Plack::Builder and prefix the mount\r\nwith a path like:\r\n```\r\nbuilder {  mount '/mypath' => 'MyApp' };\r\n```\r\nthe redirect will go to '/mypath/mypath/'"},"comment":{"url":"https://api.github.com/repos/PerlDancer/Dancer2-Plugin-Auth-Extensible/issues/comments/447660120","html_url":"https://github.com/PerlDancer/Dancer2-Plugin-Auth-Extensible/pull/81#issuecomment-447660120","issue_url":"https://api.github.com/repos/PerlDancer/Dancer2-Plugin-Auth-Extensible/issues/81","id":447660120,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0NzY2MDEyMA==","user":{"login":"abeverley","id":4589486,"node_id":"MDQ6VXNlcjQ1ODk0ODY=","avatar_url":"https://avatars0.githubusercontent.com/u/4589486?v=4","gravatar_id":"","url":"https://api.github.com/users/abeverley","html_url":"https://github.com/abeverley","followers_url":"https://api.github.com/users/abeverley/followers","following_url":"https://api.github.com/users/abeverley/following{/other_user}","gists_url":"https://api.github.com/users/abeverley/gists{/gist_id}","starred_url":"https://api.github.com/users/abeverley/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abeverley/subscriptions","organizations_url":"https://api.github.com/users/abeverley/orgs","repos_url":"https://api.github.com/users/abeverley/repos","events_url":"https://api.github.com/users/abeverley/events{/privacy}","received_events_url":"https://api.github.com/users/abeverley/received_events","type":"User","site_admin":false},"created_at":"2018-12-16T17:18:49Z","updated_at":"2018-12-16T17:18:49Z","author_association":"NONE","body":"I've just spent a while looking at this as part of another return_url problem I'm investigating. My conclusion is that the change from ```path``` to ```request_uri``` was done to encompass the full original request by the user, including things like query parameters.\r\n\r\nWhat I intend to do is change to ```path``` as suggested, but add the query parameters as well.\r\n\r\nFor reference, there is further information about path vs uri here: https://metacpan.org/pod/Plack::Request#DISPATCHING\r\n\r\nThanks for the tests. I will probably add these as a separate commit (a little tidying up is needed to remove some the extraneous copying and pasting ;-) )"}}
{"actor":{"display_login":"abeverley"}}