Project

Profile

Help

Issue #4278

Content app fails with unhandled exception, where it should 404

Added by mdellweg 10 months ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

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

Duplicated by Pulp - Issue #4321: 500 error when content path does not exist CLOSED - DUPLICATE Actions

Associated revisions

Revision 845485da View on GitHub
Added by mdellweg 9 months ago

Make content app serve 404 on wrong urls

closes #4278
https://pulp.plan.io/issues/4278

Revision 845485da View on GitHub
Added by mdellweg 9 months ago

Make content app serve 404 on wrong urls

closes #4278
https://pulp.plan.io/issues/4278

History

#1 Updated by CodeHeeler 10 months ago

  • Triaged changed from No to Yes

#2 Updated by mdellweg 10 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to mdellweg

#3 Updated by mdellweg 9 months ago

  • Status changed from ASSIGNED to MODIFIED

#4 Updated by daviddavis 9 months ago

  • Duplicated by Issue #4321: 500 error when content path does not exist added

#5 Updated by daviddavis 8 months ago

  • Tags Pulp 3 added

#6 Updated by daviddavis 6 months ago

  • Sprint/Milestone set to 3.0

#7 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF