renamed lib to tools
This commit is contained in:
parent
cdc6b571e3
commit
3bcdc49088
@ -20,7 +20,6 @@ import wtforms
|
|||||||
from sqlalchemy import or_
|
from sqlalchemy import or_
|
||||||
|
|
||||||
from mediagoblin import mg_globals
|
from mediagoblin import mg_globals
|
||||||
from mediagoblin.auth import lib as auth_lib
|
|
||||||
from mediagoblin.db.models import User
|
from mediagoblin.db.models import User
|
||||||
from mediagoblin.tools.mail import (normalize_email, send_email,
|
from mediagoblin.tools.mail import (normalize_email, send_email,
|
||||||
email_debug_message)
|
email_debug_message)
|
||||||
|
@ -16,7 +16,7 @@
|
|||||||
import uuid
|
import uuid
|
||||||
|
|
||||||
from mediagoblin.plugins.basic_auth import forms as auth_forms
|
from mediagoblin.plugins.basic_auth import forms as auth_forms
|
||||||
from mediagoblin.plugins.basic_auth import lib as auth_lib
|
from mediagoblin.plugins.basic_auth import tools as auth_tools
|
||||||
from mediagoblin.db.models import User
|
from mediagoblin.db.models import User
|
||||||
from mediagoblin.tools import pluginapi
|
from mediagoblin.tools import pluginapi
|
||||||
from sqlalchemy import or_
|
from sqlalchemy import or_
|
||||||
@ -28,7 +28,7 @@ def setup_plugin():
|
|||||||
|
|
||||||
def check_login(user, password):
|
def check_login(user, password):
|
||||||
if user.pw_hash:
|
if user.pw_hash:
|
||||||
result = auth_lib.bcrypt_check_password(password, user.pw_hash)
|
result = check_password(password, user.pw_hash)
|
||||||
if result:
|
if result:
|
||||||
return result
|
return result
|
||||||
return None
|
return None
|
||||||
@ -49,7 +49,7 @@ def create_user(registration_form):
|
|||||||
user = User()
|
user = User()
|
||||||
user.username = registration_form.username.data
|
user.username = registration_form.username.data
|
||||||
user.email = registration_form.email.data
|
user.email = registration_form.email.data
|
||||||
user.pw_hash = auth_lib.bcrypt_gen_password_hash(
|
user.pw_hash = igen_password_hash(
|
||||||
registration_form.password.data)
|
registration_form.password.data)
|
||||||
user.verification_key = unicode(uuid.uuid4())
|
user.verification_key = unicode(uuid.uuid4())
|
||||||
user.save()
|
user.save()
|
||||||
@ -65,11 +65,11 @@ def get_registration_form(request):
|
|||||||
|
|
||||||
|
|
||||||
def gen_password_hash(raw_pass, extra_salt):
|
def gen_password_hash(raw_pass, extra_salt):
|
||||||
return auth_lib.bcrypt_gen_password_hash(raw_pass, extra_salt)
|
return auth_tools.bcrypt_gen_password_hash(raw_pass, extra_salt)
|
||||||
|
|
||||||
|
|
||||||
def check_password(raw_pass, stored_hash, extra_salt):
|
def check_password(raw_pass, stored_hash, extra_salt):
|
||||||
return auth_lib.bcrypt_check_password(raw_pass, stored_hash, extra_salt)
|
return auth_tools.bcrypt_check_password(raw_pass, stored_hash, extra_salt)
|
||||||
|
|
||||||
|
|
||||||
def auth():
|
def auth():
|
||||||
@ -96,7 +96,7 @@ hooks = {
|
|||||||
'auth_get_registration_form': get_registration_form,
|
'auth_get_registration_form': get_registration_form,
|
||||||
'auth_gen_password_hash': gen_password_hash,
|
'auth_gen_password_hash': gen_password_hash,
|
||||||
'auth_check_password': check_password,
|
'auth_check_password': check_password,
|
||||||
'auth_fake_login_attempt': auth_lib.fake_login_attempt,
|
'auth_fake_login_attempt': auth_tools.fake_login_attempt,
|
||||||
'template_global_context': append_to_global_context,
|
'template_global_context': append_to_global_context,
|
||||||
('mediagoblin.plugins.openid.register',
|
('mediagoblin.plugins.openid.register',
|
||||||
'mediagoblin/auth/register.html'): add_to_form_context,
|
'mediagoblin/auth/register.html'): add_to_form_context,
|
||||||
|
@ -13,7 +13,7 @@
|
|||||||
#
|
#
|
||||||
# You should have received a copy of the GNU Affero General Public License
|
# You should have received a copy of the GNU Affero General Public License
|
||||||
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
from mediagoblin.plugins.basic_auth import lib as auth_lib
|
from mediagoblin.plugins.basic_auth import tools as auth_tools
|
||||||
from mediagoblin.tools.testing import _activate_testing
|
from mediagoblin.tools.testing import _activate_testing
|
||||||
|
|
||||||
_activate_testing()
|
_activate_testing()
|
||||||
@ -26,16 +26,16 @@ _activate_testing()
|
|||||||
|
|
||||||
def test_bcrypt_check_password():
|
def test_bcrypt_check_password():
|
||||||
# Check known 'lollerskates' password against check function
|
# Check known 'lollerskates' password against check function
|
||||||
assert auth_lib.bcrypt_check_password(
|
assert auth_tools.bcrypt_check_password(
|
||||||
'lollerskates',
|
'lollerskates',
|
||||||
'$2a$12$PXU03zfrVCujBhVeICTwtOaHTUs5FFwsscvSSTJkqx/2RQ0Lhy/nO')
|
'$2a$12$PXU03zfrVCujBhVeICTwtOaHTUs5FFwsscvSSTJkqx/2RQ0Lhy/nO')
|
||||||
|
|
||||||
assert not auth_lib.bcrypt_check_password(
|
assert not auth_tools.bcrypt_check_password(
|
||||||
'notthepassword',
|
'notthepassword',
|
||||||
'$2a$12$PXU03zfrVCujBhVeICTwtOaHTUs5FFwsscvSSTJkqx/2RQ0Lhy/nO')
|
'$2a$12$PXU03zfrVCujBhVeICTwtOaHTUs5FFwsscvSSTJkqx/2RQ0Lhy/nO')
|
||||||
|
|
||||||
# Same thing, but with extra fake salt.
|
# Same thing, but with extra fake salt.
|
||||||
assert not auth_lib.bcrypt_check_password(
|
assert not auth_tools.bcrypt_check_password(
|
||||||
'notthepassword',
|
'notthepassword',
|
||||||
'$2a$12$ELVlnw3z1FMu6CEGs/L8XO8vl0BuWSlUHgh0rUrry9DUXGMUNWwl6',
|
'$2a$12$ELVlnw3z1FMu6CEGs/L8XO8vl0BuWSlUHgh0rUrry9DUXGMUNWwl6',
|
||||||
'3><7R45417')
|
'3><7R45417')
|
||||||
@ -45,15 +45,15 @@ def test_bcrypt_gen_password_hash():
|
|||||||
pw = 'youwillneverguessthis'
|
pw = 'youwillneverguessthis'
|
||||||
|
|
||||||
# Normal password hash generation, and check on that hash
|
# Normal password hash generation, and check on that hash
|
||||||
hashed_pw = auth_lib.bcrypt_gen_password_hash(pw)
|
hashed_pw = auth_tools.bcrypt_gen_password_hash(pw)
|
||||||
assert auth_lib.bcrypt_check_password(
|
assert auth_tools.bcrypt_check_password(
|
||||||
pw, hashed_pw)
|
pw, hashed_pw)
|
||||||
assert not auth_lib.bcrypt_check_password(
|
assert not auth_tools.bcrypt_check_password(
|
||||||
'notthepassword', hashed_pw)
|
'notthepassword', hashed_pw)
|
||||||
|
|
||||||
# Same thing, extra salt.
|
# Same thing, extra salt.
|
||||||
hashed_pw = auth_lib.bcrypt_gen_password_hash(pw, '3><7R45417')
|
hashed_pw = auth_tools.bcrypt_gen_password_hash(pw, '3><7R45417')
|
||||||
assert auth_lib.bcrypt_check_password(
|
assert auth_tools.bcrypt_check_password(
|
||||||
pw, hashed_pw, '3><7R45417')
|
pw, hashed_pw, '3><7R45417')
|
||||||
assert not auth_lib.bcrypt_check_password(
|
assert not auth_tools.bcrypt_check_password(
|
||||||
'notthepassword', hashed_pw, '3><7R45417')
|
'notthepassword', hashed_pw, '3><7R45417')
|
||||||
|
Loading…
x
Reference in New Issue
Block a user