fabaff/platypush
1
0
Fork 0
forked from platypush/platypush
Code Issues Pull requests Projects Releases Packages Wiki Activity

🐛 Don't add the new password salt/iterations columns if already present.

Browse source 
And, similarly, don't remove them if they aren't on the user table.
This commit is contained in:
Fabio Manganiello 2024-05-05 21:58:51 +02:00
parent 901338e228
commit 6a8c83f99b
Signed by untrusted user: blacklight
GPG key ID: D90FBA7F76362774
1 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
platypush/migrations/alembic/versions/0876439530cb_added_password_hashing_parameters_to_.py
View file

@ -27,8 +27,11 @@ def upgrade() -> None:
print('The table `user` does not exist, skipping migration') print('The table `user` does not exist, skipping migration')
return return
op.add_column('user', sa.Column('password_salt', sa.String(), nullable=True)) if 'password_salt' not in user_table.columns:
op.add_column('user', sa.Column('hmac_iterations', sa.Integer(), nullable=True)) op.add_column('user', sa.Column('password_salt', sa.String(), nullable=True))
if 'hmac_iterations' not in user_table.columns:
op.add_column('user', sa.Column('hmac_iterations', sa.Integer(), nullable=True))
def downgrade() -> None: def downgrade() -> None:
@ -41,5 +44,8 @@ def downgrade() -> None:
print('The table `user` does not exist, skipping migration') print('The table `user` does not exist, skipping migration')
return return
op.drop_column('user', 'password_salt') if 'password_salt' in user_table.columns:
op.drop_column('user', 'hmac_iterations') op.drop_column('user', 'password_salt')
if 'hmac_iterations' in user_table.columns:
op.drop_column('user', 'hmac_iterations')