mirror of
https://github.com/r4sas/py-i2phosts
synced 2025-01-10 14:58:05 +00:00
web/lib/validation.py: call len() for "data" only once
Sort of optimisation.
This commit is contained in:
parent
28a04f9240
commit
bc3118ab44
@ -44,14 +44,15 @@ def validate_b64hash(data, check_uniq=True):
|
|||||||
"""
|
"""
|
||||||
Base64 hash validation
|
Base64 hash validation
|
||||||
"""
|
"""
|
||||||
|
length = len(data)
|
||||||
# Minimum key length 516 bytes
|
# Minimum key length 516 bytes
|
||||||
if len(data) < 516:
|
if length < 516:
|
||||||
raise forms.ValidationError('Specified base64 hash are less than 516 bytes')
|
raise forms.ValidationError('Specified base64 hash are less than 516 bytes')
|
||||||
# keys with cert may ends with anything, so check is relaxed
|
# keys with cert may ends with anything, so check is relaxed
|
||||||
if len(data) > 516 and re.match(r'[a-zA-Z0-9\-~]+$', data) == None:
|
if length > 516 and re.match(r'[a-zA-Z0-9\-~]+$', data) == None:
|
||||||
raise forms.ValidationError('Invalid characters in base64 hash')
|
raise forms.ValidationError('Invalid characters in base64 hash')
|
||||||
# base64-i2p
|
# base64-i2p
|
||||||
if len(data) == 516 and re.match(r'[a-zA-Z0-9\-~]+AA$', data) == None:
|
if length == 516 and re.match(r'[a-zA-Z0-9\-~]+AA$', data) == None:
|
||||||
raise forms.ValidationError('Invalid base64 hash')
|
raise forms.ValidationError('Invalid base64 hash')
|
||||||
if check_uniq == True:
|
if check_uniq == True:
|
||||||
# Avoid adding non-unique hashes
|
# Avoid adding non-unique hashes
|
||||||
|
Loading…
Reference in New Issue
Block a user