]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Code Review / openstack-build / neutron-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 0fc6605) | patch
Do not mask NotImplementedError exceptions
authorarmando-migliaccio <amigliaccio@nicira.com>
Wed, 3 Jul 2013 00:52:39 +0000 (17:52 -0700)
committerarmando-migliaccio <amigliaccio@nicira.com>
Wed, 3 Jul 2013 00:52:39 +0000 (17:52 -0700)
commit6ab8dd80f6c11e1e2def99bdba62b5cf706bf47e
treec6866e4a7ec48019e70451546b33a67ff7dc8121tree | snapshot
parent0fc66059afe7c1a07b078d279b7f7df15aab35eccommit | diff
Do not mask NotImplementedError exceptions

With this patch we  avoid masking NotImplementedError(s).
Previously, a catch-all clause returned 500, and it makes
sense to distinguish between genuine 500 errors (i.e. a bug),
versus 501 ones.

This opens up the issue of keeping the server behavior
consistent from one release to another, but one might argue
that this was bad design decision in the first place.

Fixes bug 1197200

Change-Id: I2d81cba5ee9e5fbe8e865378381790b5b467d255
quantum/api/v2/resource.py diff | blob | history
quantum/tests/unit/test_extensions.py diff | blob | history