implement sha256 hashing of password for security reason #7

Merged
otizonaizit merged 3 commits from romaingu/2024-heraklion-git:hash_pwd into main 2024-08-26 14:40:13 +02:00
Member
No description provided.
romaingu added 4 commits 2024-08-26 14:11:03 +02:00
Owner

you are fixing two issues simultaneously here:

  • add hashing
  • fix the password leak in add_user

can you please create two separate PRs for the two issues?

you are fixing two issues simultaneously here: - add hashing - fix the password leak in add_user can you please create two separate PRs for the two issues?
Owner

good idea to leverage the Python hashing library!

good idea to leverage the Python hashing library!
Author
Member

Hi @otizonaizit ,

can you please create two separate PRs for the two issues?
that's a good idea can you tell us how ?

Hi @otizonaizit , > can you please create two separate PRs for the two issues? that's a good idea can you tell us how ?
romaingu force-pushed hash_pwd from 33720c9b63 to de4dc255ac 2024-08-26 14:38:22 +02:00 Compare
otizonaizit merged commit 4f77cc464f into main 2024-08-26 14:40:13 +02:00
zbyszek reviewed 2024-08-26 14:42:25 +02:00
@ -32,0 +35,4 @@
m = sha256()
m.update(encoded_pwd)
return m.hexdigest()
Owner

Too many empty lines here.

Too many empty lines here.
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: ASPP/2024-heraklion-git#7
No description provided.