]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Validate name and description string
authorPranaliDeore <pranali11.deore@nttdata.com>
Mon, 11 May 2015 11:00:01 +0000 (04:00 -0700)
committerPranaliDeore <pranali11.deore@nttdata.com>
Mon, 10 Aug 2015 08:31:02 +0000 (01:31 -0700)
commita1bb185a1f0311c36fd590d48531ded6ccccc5c6
tree30e3969c55fcd78be309cc40daa945b4f60993bc
parent611608fdc9947a268789aba91306c13c404dbb64
Validate name and description string

If you pass name or description parameters with more than 255
characters to create and update apis of volume and snapshot
and create api of backup, then it returns 500 error code.

Added new method validate_name_and_description() in
cinder.api.openstack.wsgi.Controllera to validate string limit and
returned 400 if limit exceeds and also removing leading or trailing
whitespaces and string containing only whitespaces.

APIImpact
1. For all above APIs 400 response will be returned.
2. Earlier it was possible to pass only whitespaces or leading-trailing
   spaces to 'name' parameter.
   Now it will raise 400 error if only whitespaces are passed and will
   remove leading-trailing spaces if present in other cases.

Closes-Bug: 1454244
Change-Id: Iaf7159e816f69fd776a09828c3bc1d27fc9fdcdb
cinder/api/contrib/backups.py
cinder/api/openstack/wsgi.py
cinder/api/v2/snapshots.py
cinder/api/v2/volumes.py
cinder/tests/unit/api/contrib/test_backups.py
cinder/tests/unit/api/openstack/test_wsgi.py
cinder/tests/unit/api/v2/test_snapshots.py
cinder/tests/unit/api/v2/test_volumes.py