[allura] branch master updated: Fix phone.attempts_limit check, if user had surpassed it already

[brondsem]

This is an automated email from the ASF dual-hosted git repository.

brondsem pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/allura.git


The following commit(s) were added to refs/heads/master by this push:
     new b3df09680 Fix phone.attempts_limit check, if user had surpassed it 
already
b3df09680 is described below

commit b3df09680ecb141c1b80545e040ca1e41ee68a54
Author: Dave Brondsema <dbronds...@slashdotmedia.com>
AuthorDate: Mon Sep 19 13:21:09 2022 -0400

    Fix phone.attempts_limit check, if user had surpassed it already
---
 Allura/allura/lib/plugin.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Allura/allura/lib/plugin.py b/Allura/allura/lib/plugin.py
index fe2a97c52..89537d810 100644
--- a/Allura/allura/lib/plugin.py
+++ b/Allura/allura/lib/plugin.py
@@ -915,7 +915,7 @@ class ProjectRegistrationProvider:
                     'error': 'That phone number has already been used.'}
         count = user.get_tool_data('phone_verification', 'count') or 0
         attempt_limit = config.get('phone.attempts_limit', '5')
-        if count == int(attempt_limit):
+        if count >= int(attempt_limit):
             msg = 'Maximum phone verification attempts reached.'
             h.auditlog_user(msg, user=user)
             return {'status': 'error',