From: Go MAEDA <maeda@farend.jp>
Date: Mon, 9 Nov 2020 06:37:43 +0000 (+0000)
Subject: Use Setting.host_name instead of Setting.app_title as TOTP issuer to avoid name colli... 
X-Git-Tag: 4.2.0~510
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=f9ffcff997ddbce0dab415fc09915a18bc713099;p=redmine.git

Use Setting.host_name instead of Setting.app_title as TOTP issuer to avoid name collision with other instances or apps (#1237, #34234).


git-svn-id: http://svn.redmine.org/redmine/trunk@20308 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/lib/redmine/twofa/totp.rb b/lib/redmine/twofa/totp.rb
index 603661350..90b5da280 100644
--- a/lib/redmine/twofa/totp.rb
+++ b/lib/redmine/twofa/totp.rb
@@ -63,7 +63,7 @@ module Redmine
       private
 
       def totp
-        @totp ||= ROTP::TOTP.new(@user.twofa_totp_key, issuer: Setting.app_title)
+        @totp ||= ROTP::TOTP.new(@user.twofa_totp_key, issuer: Setting.host_name)
       end
     end
   end