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

Respect LIVEBOOK_BASE_URL_PATH when checking /public assets availability #2700

Conversation

elepedus
Copy link
Contributor

@elepedus elepedus commented Jul 4, 2024

This prevents the check from failing when running on a non-root path, which would otherwise make Livebook try to load the assets directly from hex.pm, which fails CORS checks.

Copy link
Member

@jonatanklosko jonatanklosko left a comment

Choose a reason for hiding this comment

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

Thanks :)

@jonatanklosko jonatanklosko marked this pull request as ready for review July 4, 2024 08:16
@jonatanklosko jonatanklosko changed the title Make cachedPublicEndpointCheck() respect LIVEBOOK_BASE_URL_PATH environment variable Respect LIVEBOOK_BASE_URL_PATH when checking /public assets availability Jul 4, 2024
@jonatanklosko jonatanklosko merged commit 834f5a1 into livebook-dev:main Jul 4, 2024
8 checks passed
jonatanklosko added a commit that referenced this pull request Jul 5, 2024
…ity (#2700)

Co-authored-by: Jonatan Kłosko <jonatanklosko@gmail.com>
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.

2 participants