kompetenzinventar/ki-backend
5
6
Fork 7
Code Issues Pull Requests 13 Packages Projects Releases 1 Activity

Authorisierung #22

Merged
weeman merged 1 commits from feature-auth into main 2021-06-28 18:12:30 +02:00
Conversation 0 Commits 1 Files Changed 7 +82 -74
weeman commented 2021-06-27 14:30:25 +02:00
Owner
Copy Link

closes #13

closes #13
weeman added 1 commit 2021-06-27 14:30:26 +02:00
implement auth
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
78c539c30a
weeman requested review from frlan 2021-06-27 14:30:31 +02:00
weeman requested review from scammo 2021-06-27 14:30:31 +02:00
frlan reviewed 2021-06-27 14:46:01 +02:00
ki/routes.py
@ -125,2 +125,4 @@
@token_auth
def update_profile(user_id):
if g.user.id != int(user_id):
return make_response({}, 403)
frlan commented 2021-06-27 14:46:01 +02:00
Owner
Copy Link

Ist das nicht mehr ein Fall für einen 400er? Die Berechtigungen sind ja da, der Request war aber "falsch" grübel

Ist das nicht mehr ein Fall für einen 400er? Die Berechtigungen sind ja da, der Request war aber "falsch" *grübel*
weeman commented 2021-06-27 14:53:08 +02:00
Author
Owner
Copy Link

¯\_(ツ)_/¯ kann man sich drüber unterhalten.

Die Berechtigung wäre "Ich darf dieses Profil bearbeiten". Das ist hier mM nicht gegeben. 400 wäre für mich eher so die Daten waren Gulasch. Z.B. für spätere Validierungen.

¯\\\_(ツ)_/¯ kann man sich drüber unterhalten. Die Berechtigung wäre "Ich darf dieses Profil bearbeiten". Das ist hier mM nicht gegeben. 400 wäre für mich eher so die Daten waren Gulasch. Z.B. für spätere Validierungen.
weeman merged commit fa8e30b299 into main 2021-06-28 18:12:30 +02:00
weeman deleted branch feature-auth 2021-06-28 18:12:33 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
scammo
frlan
Labels
Clear labels   
bug

Something is not working

  
doing

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
ready

Bereit für Release

  
review

   
security

   
wontfix

This won't be fixed

No Label
bug
doing
duplicate
enhancement
help wanted
invalid
question
ready
review
security
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 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: kompetenzinventar/ki-backend#22
No description provided.
Delete Branch "feature-auth"

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?