-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Teleport App to App CORS error #50456
Comments
It looks like you've correctly configured |
@zmb3 , General:Request URL: Response Headers:HTTP/1.1 200 OK Request HeadersOPTIONS /v2/test/auth HTTP/1.1 |
The Access-Control-Allow-Origin in the posted response above is correctly set to the requesting URL (i know its test data, but assuming it matches). Teleport sets the initial CORS preflight with the requested origin if its allowed in the spec. My cursory glance means the preflight is getting through Teleport and then failing at the destination app. Just to make sure, have you updated the CORS on the destination app to handle this origin as well? |
Another thing to consider is you may not want to force |
Expected behavior:
Teleport would proxy requests from one app to another without CORS error.
Current behavior:
Currently we are getting the following CORS error:
Access to fetch at 'https://test-auth.teleport.test.us/v2/test/auth' from origin 'https://test.teleport.test.us' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
Bug details:
Teleport version: 17.0.5
Recreation steps
Hitting the test app hits an AWS Amplify endpoint and works without issue. This then produces a login page. The actual authentication hits the test-auth app which redirects to a backend service that does SAML auth. It never gets that far. The browser kills it with the error above. Preflight does work though. I have no issues turning off CORS for these apps if possible and the easiest way forward.
Config
The text was updated successfully, but these errors were encountered: