fea: support PROBING_BASE_PATH for reverse proxy deployment#42
Merged
Conversation
When probing is deployed behind a reverse proxy with a sub-path prefix (e.g. /proxy/task-123), set PROBING_BASE_PATH to make all frontend links, API calls, and resource paths work correctly through the proxy. Server-side: - Read PROBING_BASE_PATH env var and inject window.__PROBING_BASE_PATH__ and a fetch interceptor into index.html - Rewrite src/href attributes in HTML to include base path prefix - Skip PROBING_BASE_PATH in sync_env_settings to avoid SQL parse error Frontend (WASM): - Add base_path module to read window.__PROBING_BASE_PATH__ via wasm-bindgen - Configure Dioxus WebHistory with prefix for client-side routing - Use with_base() for API URLs, sidebar logo, and file links
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
- Frontend: handle Nil/Error/TimeSeries responses in execute_query instead of throwing generic "DataFrame is Expected" error - CI: add publish input to workflow_dispatch, deploy job only runs on release or when manually toggled
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When probing is deployed behind a reverse proxy with a sub-path prefix (e.g. /proxy/task-123), set PROBING_BASE_PATH to make all frontend links, API calls, and resource paths work correctly through the proxy.
Server-side:
Frontend (WASM):