Skip to content

Commit 105888a

Browse files
authored
[BUGFIX] Redirect slash when defining ARGILLA_BASE_URL (#5796)
# Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Closes #5792 **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Bug fix (non-breaking change which fixes an issue) **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
1 parent 8e29938 commit 105888a

File tree

1 file changed

+1
-1
lines changed
  • argilla-server/src/argilla_server

1 file changed

+1
-1
lines changed

argilla-server/src/argilla_server/_app.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ def create_server_app() -> FastAPI:
171171
# This if-else clause is needed to simplify the test dependency setup. Otherwise, we cannot override dependencies
172172
# easily. We can review this once we have separate fastapi application for the api and the webapp.
173173
if settings.base_url and settings.base_url != "/":
174-
_app = FastAPI(docs_url=None, redoc_url=None, openapi_url=None, redirect_slashes=False)
174+
_app = FastAPI(docs_url=None, redoc_url=None, openapi_url=None, redirect_slashes=True)
175175
_app.mount(settings.base_url, app)
176176
return _app
177177
else:

0 commit comments

Comments
 (0)