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

Überarbeitung des Antwortverhaltens bzgl. Exceptions/Fehlermeldungen #4

Open
ChrisMalewski opened this issue May 22, 2018 · 3 comments

Comments

@ChrisMalewski
Copy link

  • Besonders vor dem Hintergrund der Sicherheit und Autorisierung, was wird zurückgeben?
  • Welche Fehlercodes?
  • Konsistent über alle Endpunkte
@ChrisMalewski ChrisMalewski changed the title Iteration über Antwortverhalten bzgl. Exceptions/Fehlermeldungen Überarbeitung des Antwortverhaltens bzgl. Exceptions/Fehlermeldungen May 22, 2018
@matthesrieke
Copy link
Member

TODO:

  • Überprüfen im Quellcode
  • WV prüft, ob fehlerhaftes Beispiel vorhanden
  • Dokumentation entsprechend erweitern (in allgemeiner API-Beschreibung)

@ChrisMalewski
Copy link
Author

  • Konsistenz zwischen Doku und Endpunkten noch nicht gegeben
  • Konsistenz über alle Endpunkte noch nicht gegeben
  • Fehlercodes sind noch zu definieren
    --> Weiterbearbeitung im nächsten Meilenstein

Unzulässige Angaben (Nutzer 25991 nicht existent) werden nicht sauber abgefangen

Folgender Endpunkt liefert gewünschtes Verhalten

Zugriff auf Subscriptions einer Gruppe, in die ich nicht reinschauen darf, da ich kein Mitglied und kein Superadmin bin:

@ChrisMalewski ChrisMalewski added the wv-tested Von WV getestet label Jul 13, 2018
@ChrisMalewski
Copy link
Author

Bei Einzelressourcen und Listen (z.B. notification/:id/subscriptions), für die ein Nutzer keine Berechtigung hat oder die nicht exisitieren --> 404 (wie github)
Bei Filterabfragen mit Filterparametern, für die der Nutzer keine Rechte hat oder nicht existieren leere Liste zurückgeben mit total: 0 und HTTP Code 200.
Exception Handling in der Doku beschreiben.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants