-
Notifications
You must be signed in to change notification settings - Fork 3.6k
feat: allow root path to run the app on /path
#14972
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
Merged
Borda
merged 24 commits into
Lightning-AI:master
from
pritamsoni-hsr:pritam/add-path-prefix
Oct 7, 2022
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
feb84f9
feat: add base path
pritamsoni-hsr 4cc4316
uvicorn fix arg
pritamsoni-hsr e836761
Merge branch 'master' into pritam/add-path-prefix
kaushikb11 ec3b0ac
Add prefix
kaushikb11 f1099c2
Fix tests
kaushikb11 385851e
update with base_path fix
pritamsoni-hsr 871c735
Merge branch 'pritam/add-path-prefix' of https://github.com/pritamson…
kaushikb11 89875c7
replace base path with root path
kaushikb11 ee3ece7
update frontend utility
kaushikb11 e9be5a7
fix function name
kaushikb11 7502fca
fix mypy
kaushikb11 31db1bf
Merge branch 'master' into pritam/add-path-prefix
kaushikb11 4c20797
fix tests
kaushikb11 8479731
Address reviews
kaushikb11 f73c23e
Address reviews
kaushikb11 d36a2d9
Address reviews
kaushikb11 688f8d2
Update tests
kaushikb11 fb0b326
Apply suggestions from code review
Borda 51b566e
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] be6e584
Update docstrong
kaushikb11 6bf63dc
Merge branch 'master' into pritam/add-path-prefix
kaushikb11 2cacac9
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 2882807
Update docstring
kaushikb11 1aa5c7a
Merge branch 'pritam/add-path-prefix' of https://github.com/pritamson…
kaushikb11 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -15,21 +15,24 @@ def __init__(self) -> None: | |||||||
self.flow: Optional["LightningFlow"] = None | ||||||||
|
||||||||
@abstractmethod | ||||||||
def start_server(self, host: str, port: int) -> None: | ||||||||
def start_server(self, host: str, port: int, root_path: str = "") -> None: | ||||||||
"""Start the process that serves the UI at the given hostname and port number. | ||||||||
|
||||||||
Arguments: | ||||||||
host: The hostname where the UI will be served. This gets determined by the dispatcher (e.g., cloud), | ||||||||
but defaults to localhost when running locally. | ||||||||
port: The port number where the UI will be served. This gets determined by the dispatcher, which by default | ||||||||
chooses any free port when running locally. | ||||||||
root_path: root_path for the server if app in exposed via a proxy at `/<root_path>` | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
You need to leave an empty line There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can we include an example like |
||||||||
|
||||||||
|
||||||||
Example: | ||||||||
|
||||||||
An custom implementation could look like this: | ||||||||
|
||||||||
.. code-block:: python | ||||||||
|
||||||||
def start_server(self, host, port): | ||||||||
def start_server(self, host, port, root_path=""): | ||||||||
self._process = subprocess.Popen(["flask", "run" "--host", host, "--port", str(port)]) | ||||||||
""" | ||||||||
|
||||||||
|
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
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
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
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
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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.