Skip to content
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

Add hubs-upload-cdn.com to NON_CORS_PROXY_DOMAINS in .defaults.env #5624

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

takahirox
Copy link
Contributor

Fixes: #5576

User upload files in hubs.local (npm run dev) are uploaded to hubs-upload-cdn.com and they are loaded via the proxy (hubs-proxy.com).

hubs-upload-cdn.com seems to have CORS that allows hubs.local then the files don't need to be loaded via the proxy.

To avoid the proxy, add hubs-upload-cdn.com to NON_CORS_PROXY_DOMAINS in .defaults.env.

User upload files in hubs.local (npm run dev) are uploaded to
hubs-upload-cdn.com and they are loaded via the proxy
(hubs-proxy.com).

hubs-upload-cdn.com seems to have CORS that allows hubs.local
then the files don't need to be loaded via the proxy.

To avoid the proxy, add hubs-upload-cdn.com to
NON_CORS_PROXY_DOMAINS in .defaults.env.
Copy link
Contributor

@brianpeiris brianpeiris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@takahirox takahirox merged commit cb20d48 into master Aug 8, 2022
@takahirox takahirox deleted the AddNonProxyDomain branch August 8, 2022 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Recorded camera videos don't play in "Open Link"
2 participants