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',
