Project

Profile

Help

Issue #4278

Content app fails with unhandled exception, where it should 404

Added by mdellweg over 1 year ago. Updated 6 months ago.

Status:
CLOSED - CURRENTRELEASE
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 over 1 year 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 over 1 year ago

Make content app serve 404 on wrong urls

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

History

#1 Updated by CodeHeeler over 1 year ago

  • Triaged changed from No to Yes

#2 Updated by mdellweg over 1 year ago

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

#3 Updated by mdellweg over 1 year ago

  • Status changed from ASSIGNED to MODIFIED

#4 Updated by daviddavis over 1 year ago

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

#5 Updated by daviddavis over 1 year ago

  • Tags Pulp 3 added

#6 Updated by daviddavis about 1 year ago

  • Sprint/Milestone set to 3.0.0

#7 Updated by bmbouter about 1 year ago

  • Tags deleted (Pulp 3)

#8 Updated by bmbouter 6 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF