Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keyresult Aktionen geben 401 im Frontend nachdem der Typ eines Keyresults geändert wurde oder ein keyresult gelöscht wurde #666

Closed
RandomTannenbaum opened this issue Dec 5, 2023 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@RandomTannenbaum
Copy link
Collaborator

Der bug
Normalerweise wenn ein Keyresult erstellt wird oder ein keyresult bearbeitet wird ohne den type zu ändern, geht dies ohne Probleme und ein toaster mit success message wird angezeigt.

Erstellen:
Image
Bearbeiten:
Image

Es scheint jedoch einen bug zu geben, durch den es beim Löschen eines Keyresults oder beim ändern eines Keyresult-types einen 401-Statuscode mit der message Du bist nicht autorisiert dieses keyresult anzuzeigen im frontend gibt und eine NoResultException mit der message No entity found for query im Backend.

Keyresult löschen:
Image
Keyresult bearbeiten mit typechange:
Image

Backend (bei beiden das Selbe):
Image

Ist dieser bug einmal aufgetreten, gibt es von nun an auch 401-Statuscodes beim erstellen und normalem Bearbeiten von keyresults. Dies ist erst nach einem site-reload wieder behoben.

Erstellen:
Image
Bearbeiten:
Image

@tuxmatta tuxmatta added the bug Something isn't working label Dec 6, 2023
@Vakmeth Vakmeth self-assigned this Dec 6, 2023
@Vakmeth
Copy link
Collaborator

Vakmeth commented Dec 6, 2023

Erkenntnis: ID des KeyResults wird nach dem PUT geändert, darauf will der Overview Call das KeyResult mit der alten ID getten, jedoch funktioniert dies nicht da die ID ja geändert wurde:

Vorgehen: Ich empfehle zu untersuchen wie man ein Ändern der ID der Entität nach dem PUT Request verhindern kann, andernfalls muss dem Frontend irgendwie die neue ID mitgeteilt werden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants