search

TeamPiped / Piped-Docker

MIT License
39 stars 32 forks source link

Update CORS headers to be more strict #5

Closed nperez0111 closed 2 years ago

nperez0111 commented 2 years ago

When running the frontend CORS was set to * on the proxy and unset on the API. Updated to use the FRONTEND_HOSTNAME instead.

I ran into this issue trying to get Piped running on my existing caddy setup. This is all I needed to change for it to work

FireMasterK commented 2 years ago

I'm not a fan of this, since it prevents the official instance switcher from working.

Won't this break that?

FireMasterK commented 2 years ago

Closing this as the headers are now added via https://github.com/FireMasterK/http3-ytproxy/blob/e615be63e45cd361a20fc9ec16384e7c6478eb17/main.go#L61-L68