implement sha256 hashing of password for security reason #7

Merged
otizonaizit merged 3 commits from :hash_pwd into main 2024-08-26 14:40:13 +02:00
No description provided.
Ghost 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!

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 ?
Ghost 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.