[FIX] Disable redirect slashes globally and remove trailing `/` from `/query`

alyssadai commented 1 month ago

Changes proposed in this pull request:

API will no longer redirect requests with or without a trailing slash to the version defined in the route decorator
/query route no longer expects a slash at the end (or before query params)
Minor fix of a leftover test context issue: authentication is now explicitly enabled for tests of request authorization headers
uvicorn command in Dockerfile updated to trust X-Forwarded... proxy headers from all remote IPs, to ensure we don't lose https from a request once behind a proxy (this should happen less with the redirects now disabled, but this change should nonetheless act as a safeguard against different proxy server setups)

Checklist

This section is for the PR reviewer

[x] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see our Contributing Guidelines for more info)
[x] PR has a label for the release changelog or skip-release (to be applied by maintainers only)
[x] PR links to GitHub issue with mention Closes #XXXX
[x] Tests pass
[x] Checks pass