Project

Profile

Help

Issue #8318

Sanitize input values in the methods implementing the pagination

Added by lmjachky 8 months ago. Updated 7 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix
Sprint:
Sprint 92
Quarter:

Description

A negative value defined for the query parameter n causes a runtime error:

(pulp) [vagrant@pulp3-source-fedora32 backup]$ http http://localhost:24817/v2/_catalog?n=-1
HTTP/1.1 500 Internal Server Error
Access-Control-Expose-Headers: Correlation-ID
Connection: close
Content-Length: 27
Content-Type: text/html
Correlation-ID: 6d0f66a77d96458bb10aacda31d673da
Date: Tue, 02 Mar 2021 14:37:36 GMT
Server: gunicorn/20.0.4
X-Frame-Options: SAMEORIGIN

<h1>Server Error (500)</h1>

Associated revisions

Revision b276dbb4 View on GitHub
Added by mdellweg 8 months ago

Sanitize page size in pagination classes

This will prevent the request to result in a 500 response when a negative page size is requested.

fixes #8318 https://pulp.plan.io/issues/8318

Revision b276dbb4 View on GitHub
Added by mdellweg 8 months ago

Sanitize page size in pagination classes

This will prevent the request to result in a 500 response when a negative page size is requested.

fixes #8318 https://pulp.plan.io/issues/8318

History

#1 Updated by lmjachky 8 months ago

  • Tracker changed from Task to Issue
  • Severity set to 2. Medium
  • Triaged set to No

#2 Updated by mdellweg 8 months ago

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

#3 Updated by pulpbot 8 months ago

  • Status changed from ASSIGNED to POST

#4 Updated by rchan 8 months ago

  • Sprint changed from Sprint 91 to Sprint 92

#5 Updated by ipanova@redhat.com 8 months ago

  • Triaged changed from No to Yes

#6 Updated by mdellweg 8 months ago

  • Status changed from POST to MODIFIED

#7 Updated by pulpbot 7 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF