Fix totp registration (openemr#6962)

* Update mfa_totp.php On the original version, if you typed your password and pressed <ENTER>, a blank screen would be displayed. This is because the <Form onsubmit=""> tag had one action, and the submit <Button> had a different onclick="" action. This change makes it so pressing <ENTER> or clicking the Submit <Button> execute the same code... * Update mfa_totp.php Put type="submit" in \<button\> * Fix TOTP Registration page * Fix use statements order --------- Co-authored-by: Beavis <[email protected]>
bradymiller · Nov 1, 2023 · f8c377c · f8c377c
1 parent a70374a
commit f8c377c
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/library/classes/Totp.class.php b/library/classes/Totp.class.php
@@ -12,6 +12,7 @@
  * @license   https://github.com/openemr/openemr/blob/master/LICENSE CNU General Public License 3
  */
 
+use RobThree\Auth\Algorithm;
 use RobThree\Auth\Providers\Qr\BaconQrCodeProvider;
 use RobThree\Auth\TwoFactorAuth;
 
@@ -60,7 +61,7 @@ public function generateQrCode()
         }
 
         $qrCodeProvider = new BaconQrCodeProvider(4, '#ffffff', '#000000', 'svg');
-        $tfa = new TwoFactorAuth($this->_issuer, 6, 30, 'sha1', $qrCodeProvider);
+        $tfa = new TwoFactorAuth($this->_issuer, 6, 30, Algorithm::Sha1, $qrCodeProvider);
         $qr = $tfa->getQRCodeImageAsDataUri($this->_username, $this->_secret);
         if (empty($qr)) {
             return false;