Add OCS API header per default

[LukasReschke]

Relieve @pvince from having to write it all the time ;-)

[scrutinizer-notifier]

The inspection completed: No new issues

[ghost]

🚀 Test PASSed. 🚀
Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/3066/
🚀 Test PASSed. 🚀

[PVince81]

Thanks... but are you sure ? There were reasons for that header ? @DeepDiver1975

I'll still need to keep them in pyocclient to keep compatible with OC 6/7.

[LukasReschke]

Thanks... but are you sure ? There were reasons for that header ? @DeepDiver1975

Yes. I am sure. I added that header 😉

It is only to to prevent being the cookies re-used from another domain. It's fine doing that here ;-)

[LukasReschke]

I'll still need to keep them in pyocclient to keep compatible with OC 6/7.

Sure. That code only adds it to all jQuery AJAX requests done from oC JS ;-)

[PVince81]

Ah yes, it was late and I didn't look at the code.
But will it affect regular ajax calls (the non-OCS ones ?), I guess it won't ?

[LukasReschke]

But will it affect regular ajax calls (the non-OCS ones ?), I guess it won't ?

It will add the header as well there, but that doesn't matter (it's simply not processed further). The CSRF token is also sent with every request even if not necessarily needed in every case.

[DeepDiver1975]

fine with me 👍

[PVince81]

👍