Issue #4278
Content app fails with unhandled exception, where it should 404
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:
Description
When trying to download from the wrong location, i'd expect the content app to serve a 404 response. Instead, it simply closes the connection.
In the test log i see:
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))
And pjournal shows:
Dec 19 10:35:10 pulp3 gunicorn[15266]: [2018-12-19 10:35:10 +0000] [15277] [ERROR] Unhandled exception
Dec 19 10:35:10 pulp3 gunicorn[15266]: Traceback (most recent call last):
Dec 19 10:35:10 pulp3 gunicorn[15266]: File "/home/vagrant/.virtualenvs/pulp/lib64/python3.6/site-packages/aiohttp/web_protocol.py", line 410, in start
Dec 19 10:35:10 pulp3 gunicorn[15266]: await resp.prepare(request)
Dec 19 10:35:10 pulp3 gunicorn[15266]: AttributeError: 'NoneType' object has no attribute 'prepare'
Related issues
Associated revisions
Revision ad6d65d3
View on GitHub
Add test that content app 404
Revision 845485da
View on GitHub
Make content app serve 404 on wrong urls
Revision 845485da
View on GitHub
Make content app serve 404 on wrong urls
History
#1
Updated by CodeHeeler about 2 years ago
- Triaged changed from No to Yes
#2
Updated by mdellweg about 2 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to mdellweg
#3
Updated by mdellweg about 2 years ago
- Status changed from ASSIGNED to MODIFIED
Applied in changeset pulp|845485daaf37360917d9b544a4c0f8dfb6d00eb8.
#4
Updated by daviddavis about 2 years ago
- Has duplicate Issue #4321: 500 error when content path does not exist added
#5
Updated by daviddavis almost 2 years ago
- Tags Pulp 3 added
#6
Updated by daviddavis almost 2 years ago
- Sprint/Milestone set to 3.0.0
#7
Updated by bmbouter almost 2 years ago
- Tags deleted (
Pulp 3)
#8
Updated by bmbouter about 1 year ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Please register to edit this issue
Add test that content app 404
re #4278 https://pulp.plan.io/issues/4278