diff options
| author | Brian Lamar <brian.lamar@rackspace.com> | 2011-03-15 19:01:27 +0000 |
|---|---|---|
| committer | Tarmac <> | 2011-03-15 19:01:27 +0000 |
| commit | d36b4d5f3797521b1c2d13a0d30fe98a0671768e (patch) | |
| tree | aebdc8c839aca8bc68ab5f02fb2e07b2fe6f34c0 /nova/utils.py | |
| parent | 568528bb0f90302106efbc4a841f05f18bd58357 (diff) | |
| parent | 909b72faa77ba0a2bc787309b95fdfae9bb9ca01 (diff) | |
| download | nova-d36b4d5f3797521b1c2d13a0d30fe98a0671768e.tar.gz nova-d36b4d5f3797521b1c2d13a0d30fe98a0671768e.tar.xz nova-d36b4d5f3797521b1c2d13a0d30fe98a0671768e.zip | |
Fixed lp732866 by catching relevant `exception.NotFound` exception. Tests did not uncover this vulnerability due to "incorrect" FakeAuthManager. I say "incorrect" because potentially different implementations (LDAP or Database driven) of AuthManager might return different errors from `get_user_from_access_key`.
Also, removed all references to 'bacon', 'ham', 'herp', and 'derp' and replaced them with hopefully more helpful terms.
Long story short it addresses the immediate issue while throughly ignoring the larger issue, which is correctly testing all implementations of Auth. I find this acceptable as currently the future of auth is in flux.
Diffstat (limited to 'nova/utils.py')
0 files changed, 0 insertions, 0 deletions