[vz-dev] JSONP exception handling

[Patrik Karisch]

CORS ist mit einer .htaccess sofort aktiviert:

Header set Access-Control-Allow-Origin "*"

JS-seitig muss nur herausgenommen werden, das JSONP verwendet wird.
Ansonsten ist das Handling wie beim normalen JSON über die selbe
Domain. Kein Unterschied. Höchstens Serverseitig muss man noch eine
OPTIONS Anfrage korrekt handlen, sollten PUT und DELETE-Requests oder
POSTs mit nicht x-www-urlencoded Daten. Da wird vom Browser ein
"Pre-Flight" ausgeführt. Siehe
https://developer.mozilla.org/en-US/docs/HTTP/Access_control_CORS#Preflighted_requests

Ansichtssache, aber beides dürfte auf die selbe Schreiberei hinauslaufen..

lg

Am 26. September 2013 16:44 schrieb Andreas Goetz <cpuidle at gmail.com>:
> Hallo Patrick,
>
> das kann alles sein. Da wir Clients mit JSON(P) haben würde ich jetzt aber
> gerne erstmal das Problem lösen. CORS zusätzlich zu implementieren ist eine
> nette Aufgabe die ich ebenfalls übernehmen würde.
>
> vg
> Andreas
>
>
>
> 2013/9/26 Patrik Karisch <patrik.karisch at gmail.com>
>>
>> Servus,
>>
>> Eine gnereller Responsecode von 200 widerspricht aber grob
>> REST-Paradigmen. Besser man wechselt auf CORS-Header Webserverseitig, dann
>> benötigt es kein JSONP mehr.
>>
>> Am 26.09.2013 13:33 schrieb "Andreas Goetz" <cpuidle at gmail.com>:
>>
>>> Hallo Zusammen,
>>>
>>> wenn in der MW eine Exception auftritt wird diese vom JSON view an den
>>> Aufrufer zurückgesandt (z.B. "invalid uuid").
>>> Bei JSONP funktioniert das nicht, weil bei HTTP Response Code 400 das
>>> JSONP Skript von remote gar nicht erst ausgeführt wird.
>>>
>>> Einen RFC für JSONP konnte ich nicht finden, daher meine Frage bevor ich
>>> einen Patch bereitstelle: sollten wir bei JSONP nicht _immer_ HTTP 200 OK
>>> zurückgeben statt 400 oder sonstiger Codes?
>>>
>>> Das Frontend wäre von dieser Änderung nicht betroffen- hier werden
>>> Exceptions auch bei HTTP 200 sauber behandelt soweit ich den Code gesehen
>>> habe.
>>>
>>> +1 für den Patch von mir- was meint Ihr?
>>>
>>> vg
>>> Andreas
>>>
>>> PS.: ich würde dann auch gerne- zusammen mit den sonstigen Änderungen-
>>> die MW Version auf 0.3 erhöhen und die entsprechende Doku im Wiki anpassen.
>>>
>



-- 
mit freundlichen Grüßen // best regards

Patrik Karisch