Changes

Talk:stoney core: REST API

4,033 bytes removed, 14:14, 11 January 2015
= Fragen / Inkonsistenzen =
 
* Wir haben im Header eine Location. Im Body manchmal ja, manchmal nein.
** Gehört die Location in die Body Response rein? Falls ja:
*** Location müsste bei Update auch nicht hochgeschrieben werden?
*** es kommt drauf an: bei einer Collection/Such-Resultat/Liste sollte bei jedem einzelnen Element eine Location mitgegeben werden unter welchem das (vollständige) Element abgerufen werden kann. Als Teil eines Elements macht es meistens keinen Sinn. Im Header sollte es nur als Antwort vom Server kommen bei einem POST-Request (also beim Erstellen einer neuen Ressource) zusammen mit einem 201, siehe [http://en.wikipedia.org/wiki/HTTP_location Wikipedia] --[[User:Tiziano|Tiziano]] ([[User talk:Tiziano|talk]]) 12:37, 5 January 2015 (CET)
 
* Update
** Statt leerem Body wäre hier eine Antwort logisch, wie bei der Kreierung.
*** Die Antwort ist gemäss REST mit dem Code 200 bereits gegeben. Vielleicht auch hier 204 verwenden stattdessen. --[[User:Tiziano|Tiziano]] ([[User talk:Tiziano|talk]]) 12:37, 5 January 2015 (CET)
 
* http://wiki.stoney-cloud.org/wiki/stoney_core:_People_Resource_-_REST_API#People_collection_retrieval_response_message
** Location ist definiert, jedoch im Example nicht drin und entsprechend auch nicht umgesetzt.
** Location würde in diesem Falle dem "current" entsprechen (zwischen "pre" und "next").
** Gemäss HTTP Protokoll auch nicht vorgesehen. Fehler in der Definition --[[User:Tiziano|Tiziano]] ([[User talk:Tiziano|talk]]) 12:37, 5 January 2015 (CET)
 
* Caching Controls: Manchmal kommt "Cache-Control: no-cache" zurück, manchmal "Cache-Control: none, private".
** ich vermute dass hier der Webserver noch was selbst macht --[[User:Tiziano|Tiziano]] ([[User talk:Tiziano|talk]]) 12:37, 5 January 2015 (CET)
 
* the service must recognize ETag, Last-Modified and Cache-Control: none provided by the client and act accordingly.
** Machen wir überhaupt etwas mit dem "Cache-Control: none"? Müsste dies nicht eh der Default-Wert sein?
 
* Mit belongsToResellerId und belongsToCustomerId sind wir inkonsequent. Manchmal haben wir eine belongsToResellerId, manchmal nicht.
* Relations (queries/scoping): Haben wir nicht getestet!
** Sonst konkret definieren, was zurück kommen muss.
** Bei einer Collection: ja. (tun wir das nicht bereits?). Bei einem Element muss ein Fehlerwert zurückgegeben werden. Ich hätte hier gerne jedoch 404 Not Found anstatt Permission Denied. --[[User:Tiziano|Tiziano]] ([[User talk:Tiziano|talk]]) 12:37, 5 January 2015 (CET)
 * Meiner Meinung nach müsste gemäss Spezifikation (On success (200) an empty response message body will be returned, otherwise ...) bei einer Löschung ein leerer JSON String und nicht "{ "success": 1 } zurück gegeben werden.** http://en.wikipedia.org/wiki/List_of_HTTP_status_codes#2xx_Success** Wir müssten eigentlich "204 No Content" erhalten:*** The server successfully processed the request, but is not returning any content. Usually used as a response to a successful delete request.** Bei "200 OK" käme sonst "In a POST request the response will contain an entity describing or containing the result of the action." zurück.*** In diesem Falle würde ich eher "{ "Deletion successful": 1 } erwarten.** Gemäss [http://stackoverflow.com/questions/2342579/http-status-code-for-update-and-delete Stackoverflow] Das ist sowohl 200 als 204 ok. Wir haben uns scheinbar für 200 entschieden. Ein Client wird vermutlich nur auf 200/204 prüfen und den Body sowieso ignorieren, von daher egal. Ich persönlich würde 204 nehmen damit wir keinen Text zurückgeben müssen. --bereits definiert [[Userstoney_core:Tiziano_REST_API#Error_codes_and_responses|Tiziano]] 403 ([[User talk:Tiziano|talkForbidden) mit einem JSON error object]]) 12:37, 5 January 2015 (CET) * Actions** Den neuen HTTP Status Code "202 Accepted" einführen?*** The request has been accepted for processing, but the processing has not been completed. The request might or might not eventually be acted upon, as it might be disallowed when processing actually takes place.*** Eine Location für die Status-Abfrage ist nötig! In Form einer Job-ID oder ähnlichem.*** Ja, aber dafür benötigen wir das Framework dahinter. Im Moment würde ich das ignorieren und so tun als wäre die Aktion sofort und erfolgreich ausgeführt wordenwelches weitere Details liefert. --[[User:TizianoChrigu|TizianoChrigu]] ([[User talk:TizianoChrigu|talk]]) 1216:3710, 5 8 January 2015 (CET)**** die Alternative dazu '''403 ist dass wir uns bald für eine Messaging Middleware Lösung für Puppet/MCollective entscheiden und darauf aufbauend einen Job Server hochbringen. Das hiesse vermutlich: [http://www.rabbitmq.com/ RabbitMQ]/[https://github.com/videlalvaro/RabbitMqBundle RabbitMQ Symfony Bundle] und [http://www.celeryproject.org/ Celery] --[[User:Tiziano|Tiziano]] ([[User talk:Tiziano|talk]]) 18:03, 7 January 2015 (CET)OK'''
* Passwörter
** Eventuell lassen wir die Vorgaben via API abfragen? In diesem Falle müsste es ins stoney-core Modul gehören?
*** Zu aufwändig für den Nutzen. --[[User:Tiziano|Tiziano]] ([[User talk:Tiziano|talk]]) 12:37, 5 January 2015 (CET)
** '''Minimale Anforderungen definieren'''
SLB, editor, reviewer
3,368
edits