auth.py: Make usage of exceptions instead of if-else-blocks #5
No reviewers
weeman
Labels
No Label
bug
doing
duplicate
enhancement
help wanted
invalid
question
ready
review
wontfix
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: kompetenzinventar/ki-backend#5
Loading…
Reference in New Issue
No description provided.
Delete Branch "auth-with-exceptions"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -18,2 +32,3 @@if auth_user["password"] != password:except (UserWrongCredentialsException, KeyError):print('Wrong username/password combination')Ich bau die Tage mal
loggingein.Was passiert bei der API mit
print? Sehe ich das dann in der Antwort? Weil es sollte irgend ein geregeltes JSON zurückkommen.Siehe #7 für Ausgaben.
@ -16,0 +22,4 @@users = yaml.safe_load(auth_file_stream)except yaml.YAMLError:print('Could not parse auth.yml.')Hier würde ich dann eine 500er Antwort zurückgeben.
Ich habe das jetzt mal versucht -- aber das ist nicht so richtig schön zu machen, wenn man nicht den kompletten Code in die routes.py verlegen will oder Datenstrutkturen von links nach rechts kopieren mag. Ich würde das mal rauslassen bis LDAP eingebaut wird. Für dev sollte das vollkommen ok sein.
Ich hab da ne Idee. Mach ich später
@ -16,0 +25,4 @@print('Could not parse auth.yml.')try:auth_user = users[username]Die Prüfung, ob der Benutzer existiert ist verschwunden oder übersehe ich was?
Ja, das ist richtig. Das wirft einen KeyError.
Pull request closed