{"issue":{"created_at":"2011-10-05T23:11:50Z","state":"open","updated_at":"2011-11-21T05:19:12Z","user":{"gravatar_id":"b226a1ac0c40f65a312c577157129a58","id":652616,"url":"https://api.github.com/users/seandowney","login":"seandowney","avatar_url":"https://secure.gravatar.com/avatar/b226a1ac0c40f65a312c577157129a58?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"comments":7,"body":"There are a couple of things about the Login Destination:\r\n\r\n- we have a setting \"auth.do_login_redirect\" which if set will use the login destination set on the role level\r\n- there is some code that if \"auth.do_login_redirect\" is not set will remember the requested url and after login bring the user to the url prior to login\r\n- at the moment if the login destination is not set on the role then it was automatically use \"/\"\r\n\r\nThis is good except in the case where \"auth.do_login_redirect\" is set but we still want one or more roles to use the requested url rather than the login destination.\r\n\r\nUsing the \"requested url\" is probably the preferred option on the Front end and the \"login destination\" on the backend.\r\n\r\nI think that even if \"auth.do_login_redirect\" is set but the role \"login destination\" is not set then it should use the \"requested url\" rather than \"/\" as a default.\r\n\r\nWhat do you think?\r\n","assignee":{"gravatar_id":"b226a1ac0c40f65a312c577157129a58","id":652616,"url":"https://api.github.com/users/seandowney","login":"seandowney","avatar_url":"https://secure.gravatar.com/avatar/b226a1ac0c40f65a312c577157129a58?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"id":1829596,"number":232,"labels":[{"name":"0.3","color":"006688","url":"https://api.github.com/repos/ci-bonfire/Bonfire/labels/0.3"},{"name":"Enhancement","color":"0b02e1","url":"https://api.github.com/repos/ci-bonfire/Bonfire/labels/Enhancement"}],"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"title":"Login Destination anomaly ","html_url":"https://github.com/ci-bonfire/Bonfire/issues/232","closed_at":null,"milestone":{"created_at":"2011-06-02T13:41:58Z","state":"open","due_on":"2012-01-03T08:00:00Z","closed_issues":12,"creator":{"gravatar_id":"5f29875af9d4c913fd01bf426ddcdc51","id":697249,"url":"https://api.github.com/users/ci-bonfire","login":"ci-bonfire","avatar_url":"https://secure.gravatar.com/avatar/5f29875af9d4c913fd01bf426ddcdc51?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"number":3,"title":"0.3","description":"0.3's primary focuses are on more flexible users, better security by implementing a settings table and allowing all files other than assets and index.php to be moved out of root, and a UI refresh.","open_issues":25,"url":"https://api.github.com/repos/ci-bonfire/Bonfire/milestones/3"},"url":"https://api.github.com/repos/ci-bonfire/Bonfire/issues/232"},"action":"created","comment":{"created_at":"2011-11-21T05:19:12Z","updated_at":"2011-11-21T05:19:12Z","user":{"gravatar_id":"69b0b75f21d3d55f0bf99d800a1e4547","id":51931,"url":"https://api.github.com/users/lonnieezell","login":"lonnieezell","avatar_url":"https://secure.gravatar.com/avatar/69b0b75f21d3d55f0bf99d800a1e4547?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"Unfortunately it was, but I'm just about back up and running at full speed, thanks to Time Machine.\r\n\r\nI think there should definitely be a migration for this. I don't know how many times I've been frustrated trying to log into a site only to have to find the page I really wanted to go to. :) So let's help clean up our own little corners of the web for one less frustration.","id":2811002,"url":"https://api.github.com/repos/ci-bonfire/Bonfire/issues/comments/2811002"}}