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.

Sign up for GitHub

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

Jump to bottom

Add support for ASGI root_path for openapi docs #1199

Merged
merged 11 commits into from
Jun 11, 2020
Merged

Add support for ASGI root_path for openapi docs #1199

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
5fad173
Use ASGI root_path when it is provided and openapi_prefix is empty.
iksteen Mar 31, 2020
edd212d
Strip trailing slashes from root_path.
iksteen Mar 31, 2020
79d056a
Please mypy.
iksteen Mar 31, 2020
230c0a3
Fix extending openapi test.
iksteen Mar 31, 2020
7138eee
📝 Add docs and tutorial for using root_path behind a proxy
tiangolo Jun 11, 2020
dc4c05c
♻️ Refactor application root_path logic, use root_path, deprecate ope…
tiangolo Jun 11, 2020
07d7c37
✅ Add tests for Behind a Proxy with root_path
tiangolo Jun 11, 2020
cab212f
♻️ Refactor test
tiangolo Jun 11, 2020
1cc1d31
📝 Update/add docs for Sub-applications and Behind a Proxy
tiangolo Jun 11, 2020
8e2f059
📝 Update Extending OpenAPI with openapi_prefix parameter
tiangolo Jun 11, 2020
ac517f0
✅ Add test for deprecated openapi_prefix
tiangolo Jun 11, 2020
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
Prev Previous commit
Next Next commit
Please mypy.
  • Loading branch information
@iksteen
iksteen committed Mar 31, 2020
commit 79d056a071567964c869a66482a64df41be9ff6b
4 changes: 2 additions & 2 deletions fastapi/applications.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,9 +87,9 @@ def __init__(
self.openapi_schema: Optional[Dict[str, Any]] = None
self.setup()

def get_openapi_prefixed(self, req: Request, path: str = "") -> str:
def get_openapi_prefixed(self, req: Request, path: Optional[str] = "") -> str:
prefix = self._openapi_prefix or req.scope.get("root_path", "").rstrip("/")
return prefix + path
return prefix + (path or "")

def openapi(self, openapi_prefix: str) -> Dict:
if not self.openapi_schema:
Expand Down