coralproject / talk

A better commenting experience from Vox Media
https://coralproject.net
Other
1.89k stars 355 forks source link

OpenID Connect HTTP Callback Url #4507

Closed knom closed 7 months ago

knom commented 7 months ago

We have CORAL running on an HTTPS instance of Fargate in AWS.

Expected behavior: Using Docker installation guide at https://docs.coralproject.net/, Redirect URI in OpenID Conenct login is having http instead of https i.e.

http://talk.sitename.com/api/auth/oidc/callback instead of https://talk.sitename.com/api/auth/google/callback

Actual behavior: OpenID Connect redirect URI should include https instead of http The Config URL Textbox shows the hardcoded http://... value - and it cannot be changed.

Related Issues: Similar to #3667 - but for OpenID Connect.

Versions: Using latest version of Docker installation

losowsky commented 7 months ago

Thanks - well take a look.

nick-funk commented 7 months ago

We do not support serverless hosting or network stacks with Coral. Our system reconstructs secure and insecure url's when it finds tenants and other requests here: https://github.com/coralproject/talk/blob/develop/server/src/core/server/app/url.ts#L33

The reason you're likely seeing http instead of https is that somewhere in the network stack in front of your Coral pods (load balancer, reverse proxy), it is stripping the https when it redirects the requests to Coral.

knom commented 7 months ago

Hey @nick-funk: I have a question regarding support.

Thanks for your help :-)