Defect #6438

Autologin cannot be disabled again once it's enabled

Added by Anonymous almost 12 years ago. Updated almost 12 years ago.

Status:ClosedStart date:2010-09-20
Priority:LowDue date:
Assignee:Felix Schäfer% Done:


Target version:1.0.3
Resolution:Fixed Affected version:1.0.1


We have enabled the AutoLogin feature so that people don't have to enter their credentials each time they go to Redmine. When playing with it, I realized that it cannot be deactivated again. It save the settings page without giving any error but does not change the field. I also checked and it really is not deactivated.

It is possible to change the lease time though.

See also this discussion

Associated revisions

Revision 4306
Added by Jean-Baptiste Barth almost 12 years ago

Fixed: disabling autologin not persisted. #6438

Contributed by Felix Schäfer


#1 Updated by Felix Schäfer almost 12 years ago

  • Target version set to 1.0.3

JB, Eric: the commit for this is here:

The change is small: it forces the select list to return 0 instead of '' for "autologin disabled". The problem with having an empty string returned for autologin is that autologin is defined as an int in the settings and thus checked for numericality, which fails for the empty string.

Jan, if you feel up to applying a patch to your installation, here's the diff:

diff --git a/app/views/settings/_authentication.rhtml b/app/views/settings/_authentication.rhtml
index da29bb7..1122053 100644
--- a/app/views/settings/_authentication.rhtml
+++ b/app/views/settings/_authentication.rhtml
@@ -3,7 +3,7 @@
 <div class="box tabular settings">
 <p><%= setting_check_box :login_required %></p>

-<p><%= setting_select :autologin, [1, 7, 30, 365].collect{|days| [l('datetime.distance_in_words.x_days', :count => days), days.to_s]}, :blank => :label_disabled %></p>
+<p><%= setting_select :autologin, [[l(:label_disabled), 0]] + [1, 7, 30, 365].collect{|days| [l('datetime.distance_in_words.x_days', :count => days), days.to_s]} %></p>

 <p><%= setting_select :self_registration, [[l(:label_disabled), "0"],
                                                                                     [l(:label_registration_activation_by_email), "1"],

#2 Updated by Anonymous almost 12 years ago

Many thanks for your efforts! I will wait for that version, it's not really a work hindering bug, I just noticed it and thought it might be worth reporting. Thanks for being so superquick.

#3 Updated by Jean-Baptiste Barth almost 12 years ago

  • Status changed from New to Resolved
  • Resolution set to Fixed

Applied in r4306, thanks.

#4 Updated by Eric Davis almost 12 years ago

  • Status changed from Resolved to Closed

Merged into 1.0-stable for release in 1.0.3

Also available in: Atom PDF