]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix ratelimiting
authorSeif Lotfy <s.lotfy@telekom.de>
Fri, 9 Aug 2013 13:37:38 +0000 (13:37 +0000)
committerSeif Lotfy <s.lotfy@telekom.de>
Fri, 9 Aug 2013 21:18:11 +0000 (21:18 +0000)
commit2acb6f329bb404e224e525b946c0aa6cdc74040c
treeb118958c7172755a4622e013b0b6669ad627390d
parent94c720c71d70e2c428abc597268ffd053307694d
Fix ratelimiting

Current master does now respect ratelimiting, since parsing of the
api-paste.ini was faulty. api-paste.ini limited user limiting by
setting a line as follows:
user:<user-id>:(GET, *, ".*", 4, minute) which was passed to the
Limiter as kwargs with "user" as a key. Thus multiple user limiting
was not possible as well as extracting the id of the user was bound
to fail, since we checked on the key with startswith("user:")

An example config in the api-paste.ini has to look as follows:
limits = (POST, "*", .*, 10, MINUTE)
limits.<user-id1>:(GET, "*", .*, 4, minute)
limits.<user-id2>:(GET, "*", .*, 2, minute)

Fixes bug: 1206976

Change-Id: I4adbe3dbe3a0bd607d6e675f230b0442b08ec791
cinder/api/v1/limits.py
cinder/api/v2/limits.py
cinder/tests/api/v1/test_limits.py
cinder/tests/api/v2/test_limits.py