Redmine

I subscribe to my activity view via activity.atom (using feedly). Links to e.g. wiki pages or commits are in the form https://my.site/redmine/login?back_url=https%3A%2F%2Fmy.site%2Fredmine%2Fprojects%2Fmyproject%2Frepository%2Frevisions%2F3hash

I work in the 2.4-stable branch, but looking at the code it seems that trunk has the same issue.

Bug:
When clicking on these links (or pasting them to the browser), accounts#login is executed. If I am already logged in, this executes redirect_to home_url (app/controllers/account_controller.rb:37) and I end up on the Redmine start page (/redmine). The back_url parameter is ignored.

Solution:

Index: app/controllers/account_controller.rb
===================================================================
--- app/controllers/account_controller.rb       (revision 12689)
+++ app/controllers/account_controller.rb       (working copy)
@@ -34,7 +34,7 @@
   def login
     if request.get?
       if User.current.logged?
-        redirect_to home_url
+        redirect_back_or_default home_url
       end
     else
       authenticate_user