-
Notifications
You must be signed in to change notification settings - Fork 2.8k
bundle console assets into server (close #516, close #521, close #2130) #2192
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
…nto console-assets
|
Deploy preview for hasura-docs ready! Built with commit 302adba |
0x777
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Server side changes LGTM
rikinsk-zz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved (crossing fingers)
|
Review app https://hge-ci-pull-2192.herokuapp.com is deleted |
|
@shahidhk Based on this PR we need download asset file or use another docker image? |
…close hasura#2130) (hasura#2192) This PR builds console static assets into the server docker image at `/srv/console-assets`. When env var `HASURA_GRAPHQL_CONSOLE_ASSETS_DIR=/srv/console-assets` or flag `--console-assets-dir=/srv/console-assets` is set on the server, the files in this directory are served at `/console/assets/*`. The console html template will have a variable called `cdnAssets: false` when this flag is set and it loads assets from server itself instead of CDN. The assets are moved to a new bucket with a new naming scheme: ``` graphql-engine-cdn.hasura.io/console/assets/ /common/{} /versioned/<version/{} /channel/<channel>/<version>/{} ``` Console served by CLI will still load assets from CDN - will fix that in the next release.
…ip (#2192) The LHS headers were getting lost, keep them. V3_GIT_ORIGIN_REV_ID: 1dea4b42fb66a39700cab2559d171e7bc11cba8b
This PR builds console static assets into the server docker image at
/srv/console-assets. When env varHASURA_GRAPHQL_CONSOLE_ASSETS_DIR=/srv/console-assetsor flag--console-assets-dir=/srv/console-assetsis set on the server, the files in this directory are served at/console/assets/*.The console html template will have a variable called
cdnAssets: falsewhen this flag is set and it loads assets from server itself instead of CDN.The assets are moved to a new bucket with a new naming scheme:
Console served by CLI will still load assets from CDN - will fix that in the next release.