Skip to content

deposit: Fix handling of SCRIPT_NAME with gunicorn 22.0

Antoine Lambert requested to merge anlambert/docker:fix-deposit-reverse-proxy-config into master

There is no /deposit path prefix in deposit server URLs so requesting http://nginx/deposit/1/.* was returning a 404 HTTP error.

gunicorn 22.0 silently discard HTTP headers with an underscore in it so the SCRIPT_NAME header is no longer honored.

Set the SCRIPT_NAME environment variable in the swh-deposit service instead to restore previous behavior.

This fixes the deposit test errors reported in https://jenkins.softwareheritage.org/job/DENV/job/docker-tests/331/console.

Why these tests were not failing previously is still a mystery to me.

Nevertheless, these new test failures seems related to the freshly released swh.deposit v2.2.1.

Edited by Antoine Lambert

Merge request reports