Hydriz has uploaded a new change for review.
https://gerrit.wikimedia.org/r/321168
Change subject: Declare issuer name in QR code when setting up 2FA
......................................................................
Declare issuer name in QR code when setting up 2FA
The issuer name is an optional but important feature that allows
the user to differentiate between different accounts used in the
same authenticator app. While we currently use a prefix in the
user account name, declaring an issuer makes it easier for the
user to differentiate.
Bug: T150596
Change-Id: I741dd671e79e0326dfe97bdaaf63b3997960d115
---
M OATHUser.php
M special/SpecialOATHEnable.php
2 files changed, 13 insertions(+), 5 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/OATHAuth
refs/changes/68/321168/1
diff --git a/OATHUser.php b/OATHUser.php
index e5822c0..10a6312 100644
--- a/OATHUser.php
+++ b/OATHUser.php
@@ -31,13 +31,19 @@
/**
* @return String
*/
- public function getAccount() {
+ public function getIssuer() {
global $wgSitename, $wgOATHAuthAccountPrefix;
-
if ( $wgOATHAuthAccountPrefix !== false ) {
- return
"$wgOATHAuthAccountPrefix:{$this->user->getName()}";
+ return $wgOATHAuthAccountPrefix;
}
- return "$wgSitename:{$this->user->getName()}";
+ return $wgSitename;
+ }
+
+ /**
+ * @return String
+ */
+ public function getAccount() {
+ return $this->user->getName();
}
/**
diff --git a/special/SpecialOATHEnable.php b/special/SpecialOATHEnable.php
index ecc26a1..86f5aee 100644
--- a/special/SpecialOATHEnable.php
+++ b/special/SpecialOATHEnable.php
@@ -83,7 +83,9 @@
$qrcodeUrl = "otpauth://totp/"
. rawurlencode( $this->OATHUser->getAccount() )
. "?secret="
- . rawurlencode( $secret );
+ . rawurlencode( $secret )
+ . "&issuer="
+ . rawurlencode( $this->OATHUser->getIssuer() );
$qrcodeElement = Html::element( 'div', [
'data-mw-qrcode-url' => $qrcodeUrl,
--
To view, visit https://gerrit.wikimedia.org/r/321168
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I741dd671e79e0326dfe97bdaaf63b3997960d115
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/OATHAuth
Gerrit-Branch: master
Gerrit-Owner: Hydriz <hyd...@jorked.com>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
