]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix 401 from auth_token middleware
authorVishvananda Ishaya <vishvananda@gmail.com>
Wed, 7 Nov 2012 04:12:00 +0000 (20:12 -0800)
committerVishvananda Ishaya <vishvananda@gmail.com>
Wed, 7 Nov 2012 04:17:12 +0000 (20:17 -0800)
commit4e619fbef8671697bfec8e752818944c9f96cd72
tree0b728781b5a82dff940a25489b4ee22453543e73
parent5aa1e72911ef690c64c670c6605a189d7d11ae29
Fix 401 from auth_token middleware

The openssl popen call to verify the token returns an error code
of 2 when a file doesn't exist. Normally this is caught and the
relevant file is downloaded, but monkeypatching os causes this
error code to be eaten and cinder to return a 401. Note that this
is likely a race condition that depends on exactly when greenthreads
are switching that is causing the return code from Popen to get
overwritten.

It is possible that this is exposing an underlying bug in eventlet
but the workaround is simply to make sure that we don't monkeypatch
os.

Fixes bug 1075838

Change-Id: I69ba75136e32e95c6fdf108f0c4fe21a5e3bdbe6
bin/cinder-all
bin/cinder-api
bin/cinder-scheduler
bin/cinder-volume