Merge lp:~corey.bryant/ubuntu/utopic/python-keystonemiddleware/1.0.0-1ubuntu1 into lp:ubuntu/utopic-proposed/python-keystonemiddleware

Proposed by Corey Bryant on 2015-06-15
Status: Merged
Merge reported by: Martin Pitt
Merged at revision: not available
Proposed branch: lp:~corey.bryant/ubuntu/utopic/python-keystonemiddleware/1.0.0-1ubuntu1
Merge into: lp:ubuntu/utopic-proposed/python-keystonemiddleware
Diff against target: 45 lines (+27/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/refresh-expired-admin-token.patch (+18/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~corey.bryant/ubuntu/utopic/python-keystonemiddleware/1.0.0-1ubuntu1
Reviewer Review Type Date Requested Status
Martin Pitt 2015-06-15 Approve on 2015-06-26
Review via email: mp+262010@code.launchpad.net
To post a comment you must log in.
Martin Pitt (pitti) wrote :

Apparently this was already uploaded, closing.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2014-07-08 14:25:47 +0000
3+++ debian/changelog 2015-06-15 19:42:24 +0000
4@@ -1,3 +1,11 @@
5+python-keystonemiddleware (1.0.0-1ubuntu1) UNRELEASED; urgency=medium
6+
7+ * d/p/refresh-expired-admin-token.patch: Fix bug to enable refresh of
8+ expired admin token. Thanks to Sergio Martins for recommending
9+ this fix. (LP: #1460833).
10+
11+ -- Corey Bryant <corey.bryant@canonical.com> Mon, 15 Jun 2015 15:13:31 -0400
12+
13 python-keystonemiddleware (1.0.0-1) unstable; urgency=medium
14
15 * Initial release. (Closes: #755135)
16
17=== added file 'debian/patches/refresh-expired-admin-token.patch'
18--- debian/patches/refresh-expired-admin-token.patch 1970-01-01 00:00:00 +0000
19+++ debian/patches/refresh-expired-admin-token.patch 2015-06-15 19:42:24 +0000
20@@ -0,0 +1,18 @@
21+Description: Fix bug to enable refresh of expired admin token.
22+ https://bugs.launchpad.net/ubuntu/+source/python-keystonemiddleware/+bug/1460833
23+ Thanks to Sergio Martins for recommending this fix.
24+Author: Corey Bryant <corey.bryant@canonical.com>
25+Forwarded: No
26+Last-Update: 2015-06-15
27+
28+--- a/keystonemiddleware/auth_token.py
29++++ b/keystonemiddleware/auth_token.py
30+@@ -1119,7 +1119,7 @@
31+ raise InvalidUserToken('Token authorization failed')
32+ if response.status_code == 401:
33+ self._LOG.info('Keystone rejected admin token, resetting')
34+- self.admin_token = None
35++ self._admin_token = None
36+ else:
37+ self._LOG.error('Bad response code while validating token: %s',
38+ response.status_code)
39
40=== modified file 'debian/patches/series'
41--- debian/patches/series 2014-07-08 14:25:47 +0000
42+++ debian/patches/series 2015-06-15 19:42:24 +0000
43@@ -1,1 +1,2 @@
44 removes-discover-from-test-requirements.txt
45+refresh-expired-admin-token.patch

Subscribers

People subscribed via source and target branches

to all changes: