openstack/swift
Code Issues Proposed changes

Bring "egregious range request" comment in line with reality

Browse Source 
With MAX_RANGES pulled out as a constant, developers are likely to
just trust the comment rather than go verify the value.

Change-Id: I47fbd4790492b272d53ee07fe5bc87170ce2c72b
Releated-Change: I4dcedcaae6c3deada06a0223479e611094d57234
This commit is contained in:
Tim Burke 2016-09-12 15:02:38 -07:00
parent 7a166cccfb
commit f35829c291
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
swift/common/swob.py
View File

@ -586,7 +586,7 @@ class Range(object):
#
# We're defining "egregious" here as:
#
# * more than 100 requested ranges OR
# * more than 50 requested ranges OR
# * more than 2 overlapping ranges OR
# * more than 8 non-ascending-order ranges
if len(all_ranges) > MAX_RANGES: