ICRAR / ngas

69%

master: 69%

LAST BUILD ON BRANCH NGAS-163

branch: NGAS-163

CHANGE BRANCH

x

Reset

• NGAS-163
• master
• v11
• next
• macos-hostnames
• smtp-testing
• plugins-importing
• clone-order-fixes
• clone-fix
• various_improvements
• volumes
• none-reg-parameters
• relaxed-netifaces
• fs-monitor
• subscription_thread_ng
• macos-travis-fix
• bbcp_fixes
• fits_optional_arcfile
• travis-fixes
• coverage-stability
• c_client_warning_fixes
• centos7-compatibility-changes
• fs-monitor-fixes
• small-improvements
• no-create-venv
• python-client-improvements
• doc-updates
• netifaces-fix
• janitor-thread-startup
• stream-proxy-request
• separate-test-servers
• partner_sites
• travis-errors
• github-actions
• avoid-installing-bsddb3-in-py27
• subscription-def
• macos-travis-update
• issue-38
• aiosmtpd
• issue-13
• correct-logging-c
• issue-44
• improve-register-error-message
• remove-test-generic-registration-plugin
• issue-50
• issue-51
• servers-flag
• update-travis-matrix
• py39
• correct-janitor-init
• travis-focal
• uwe-fixes
• requests-pinning
• fix-requests-dependency-specs
• 98-query-ignore
• release-12.0
• v12.0
• v12
• document-future-removal-of-27
• remove-2.7-builds
• issue-103
• py311
• richer-archiving-event
• docs-fix
• direct_sql

Return 411 when on invalid/missing Content-Length

As stated in NGAS-163, NGAS should respond with 411 rather than 400 when
requests with no Content-Length (or Content-Length=0) arrive, as the
codebase currently expects a Content-Length to be given.

To keep things consistent, instead of sending the response back directly
where the problem is identified, we instead simply raise a typed error,
which we then handle more centrally with the rest of the known errors.

The unit test checking that zero-length files cannot be archived has
been updated to double-check the HTTP status from the failing request.
Adding this check also made part of test_QArchive obsolete and
duplicate (it does exactly the same that test_FileSizeZero does), so I
removed the duplicate.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>