kompetenzinventar/ki-backend
5
6
Fork 4
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 1 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
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Brain drone-bot frlan gulliver igk Lasagne marei Nikolai renovate-bot scammo srsh weeman
No Assignees
2 Participants
Notifications
Due Date
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?