-
-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug fix for 2FA. #1882
Bug fix for 2FA. #1882
Conversation
$expSeconds = $prefix == "" ? $this->dbSettings->getExpiringSeconds() : | ||
($prefix == "#" ? $this->dbSettings->getExpiringSeconds() : | ||
($prefix == "+" ? $this->dbSettings->getExpiringSeconds() : | ||
($prefix == "=" ? $this->dbSettings->getExpiringSeconds2FA() : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
== による比較の部分、 === にすることは可能でしょうか?
PHP 8.0で比較演算子の扱いが変わっているので、可能なら === にしておくとよいと思います。
https://www.php.net/manual/ja/language.operators.comparison.php
より
警告
PHP 8.0.0 より前のバージョンでは、 文字列が数値または数値形式の文字列の場合、文字列は比較する前に数値に変換されていました。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
なるほど...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ご対応ありがとうございました!
Collectedly I fixed some bugs that were detected by unit test. |
No description provided.