]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Pop out 'offset' and 'limit' before use for filter
authorxiaoxi_chen <xiaoxi.chen@intel.com>
Mon, 29 Jul 2013 06:21:25 +0000 (14:21 +0800)
committerxiaoxi_chen <xiaoxi.chen@intel.com>
Tue, 30 Jul 2013 05:16:18 +0000 (13:16 +0800)
commit08af981ffc104d7cb836c1b674e1d73724eeaf88
treebbdbd5eea1192e553cd6aab9e92d39581892e620
parent6f762cf926468cd6b47c8a49f07b2bbdd18e57d6
Pop out 'offset' and 'limit' before use for filter

In previous code of _items() from api/v{1,2}/snapshots.py,
and also the _items)_ from api/v1/volume.py.we didn't pop
out the 'offset' and 'limit' fields from HTTP get params
before we use such params for filter.This is the root cause
for bug #1205956

For non-admin user, since 'offset' and 'limit' is not in the
allowed_search_options, so the volumes.remove_invalid_options
will help to filter them out. As a result, it walks around this
bug.

But for admin user,the volumes.remove_invalid_options will not
try to filter the search_options.So for admin user, the 'limit'
will appear in search_options, then obviously get no result.

fixed bug #1205956

Change-Id: Ib1a66c9d104ac52d6eae18be7f06d02985d4c2fd
cinder/api/v1/snapshots.py
cinder/api/v1/volumes.py
cinder/api/v2/snapshots.py
cinder/tests/api/v1/test_snapshots.py
cinder/tests/api/v1/test_volumes.py
cinder/tests/api/v2/test_snapshots.py
cinder/tests/api/v2/test_volumes.py