HapiJs CORS doesn't respect pre-flight for route-based conf

CoderDojo / community-platform

Zen, the CoderDojo Community Platform!

https://zen.coderdojo.com

MIT License

121 stars 55 forks source link

HapiJs CORS doesn't respect pre-flight for route-based conf #1144

Closed Wardormeur closed 6 years ago

Wardormeur commented 7 years ago

https://github.com/hapijs/hapi/issues/2840 Updating hapijs requires to update others plugins as well as NodeJs (we shouldn't even use it on 0.10). To avoid preflight for non-cors calls (/api/2.0/dojos), avoid sending extra-headers for those routes

Wardormeur commented 6 years ago

Need to be verified if https://github.com/CoderDojo/cp-zen-platform/pull/1186 solved this

Wardormeur commented 6 years ago

Fixed with https://github.com/CoderDojo/cd-theme/pull/73