Skip to content

Conversation

dacosta444
Copy link
Contributor

@dacosta444 dacosta444 commented Aug 4, 2025

Based on the work in PR #7816

"When attempting to run Cube behind a reverse proxy, it is only possible to set the base path for the API endpoints of Cube. This renders the playground unusable inside a non root base path.

For example when using /cube as a base path behind a reverse proxy, I can get the index.html to load, from /cube/index.html but then requests for assets are made to /some-asset.js instead of /cube/some-asset.js"

In addition to the changes made in that PR the requests to playground were still using the absolute path.
Fonts were also using an absolute path and the fix for that was forcing a relative path.

Check List

  • Tests have been run in packages where changes made if available
  • Linter has been run for changed code
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

Issue Reference this PR resolves
#3028 #7969

@dacosta444 dacosta444 requested a review from a team as a code owner August 4, 2025 17:56
@github-actions github-actions bot added the pr:community Contribution from Cube.js community members. label Aug 4, 2025
@jpsfs
Copy link

jpsfs commented Aug 4, 2025

This is great work!
Hope this gets merged soon! 🙏

@dacosta444
Copy link
Contributor Author

Would it be possible to trigger the action to build and publish a Docker image for this PR so I can use it while the review is still pending?

It would be super helpful to have a published image using the GitHub Actions flow — something similar to the build and publish step
cc @vasilev-alex, @ovr
Thanks a lot in advance!

@igorlukanin igorlukanin requested a review from tenphi August 7, 2025 16:18
@tenphi tenphi merged commit 197ba2a into cube-js:master Aug 13, 2025
1 check was pending
@tenphi
Copy link
Member

tenphi commented Aug 13, 2025

@dacosta444 Sorry for the long wait.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:community Contribution from Cube.js community members.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants