ASPP/2024-heraklion-git
9
0
Fork 2
Code Issues 2 Pull requests 1 Projects Releases Packages Wiki Activity Actions

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
Conversation 4 Commits 3 Files changed 1 +14 -5
Ghost commented 2024-08-26 14:11:02 +02:00
Copy link
No description provided. 
# doc.python.org
otizonaizit commented 2024-08-26 14:15:25 +02:00
Owner
Copy link

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?
otizonaizit commented 2024-08-26 14:19:23 +02:00
Owner
Copy link

good idea to leverage the Python hashing library!

good idea to leverage the Python hashing library!
Ghost commented 2024-08-26 14:20:36 +02:00
Copy link

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 ?
zbyszek reviewed 2024-08-26 14:42:25 +02:00
auth.py
@ -32,0 +35,4 @@
m = sha256()
m.update(encoded_pwd)
return m.hexdigest()
zbyszek commented 2024-08-26 14:39:18 +02:00
Owner
Copy link

Too many empty lines here.

Too many empty lines here.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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.