-
Notifications
You must be signed in to change notification settings - Fork 881
feat: allow cross-origin requests between users' own apps #7688
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
Conversation
Originally I also overwrote CORS-related headers from the application (if it set any) but I ended up removing that because: a) Currently users can already set their own headers (aside from OPTIONS requests) so this would be adding a new restriction I am not yet sure we want. |
Because we issue auth cookies based on each subdomain, I think we are going to get 403 forbidden on these CORs requests unless the user authenticated to the other domain first. But we can handle that later, just going to crop up quick 😢 |
Ahh good point, tested to confirm this is indeed a problem, the requests get redirected to auth.
Yeah I will merge this in for now, I have no idea how we are going to solve it though. Maybe we can at least return an error on cross-origin requests that says "you have to go authenticate first". Or if we can somehow authenticate without the redirection. |
I think it was lost when I was resolving a conflict here.
Closes #5706