rowe42 / lhm_animad_admin_html5

0 stars 6 forks source link

cors statt no-cors #158

Closed xdoo closed 6 years ago

xdoo commented 6 years ago

Es wurde bei den fetch Aufrufen die Methode die Request Methode von cors auf no-cors umgestellt. Das führt zu den Problemen, die in den Tickets #149, #150 und #153 beschrieben sind. Die Stellen die zu ändern sind:

https://github.com/xdoo/lhm_animad_admin_html5/blob/7c122a2130be07b73196a7a539c26ed4c3275512/src/behaviors/animad-form-behavior.html#L137-L142

und

https://github.com/xdoo/lhm_animad_admin_html5/blob/7c122a2130be07b73196a7a539c26ed4c3275512/src/behaviors/animad-form-behavior.html#L192-L198

Aktuell brauchen wir cors. Ich denke mittelfristig werden wir auf same-origin umstellen wollen.

MDN Web Docs sagt dazu:

Rein gekommen ist das mit diesem commit: d102725bc62d03470586c97a0c606f9187937dc6

@rowe42 ich weise dir das Ticket mal zu :)

rowe42 commented 6 years ago

Warum brauchen wir cors? Wir sollten nur genau eine Domain aufrufen, und zwar die vom API-Gateway.

rowe42 commented 6 years ago

OK, habe wieder zu no-cors geändert, damit die mockables noch funktionieren.