Apache
/
jclouds
mirror of https://github.com/apache/jclouds.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Revert "A 401 error should result in re-authenticating for a new token" 

This reverts commit 833f046a5c.

References JCLOUDS-231
This commit is contained in:
Andrew Gaul 2013-08-05 21:31:49 -07:00
parent f3df06aff0
commit b2bf75f8ef
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
apis/openstack-keystone/src/main/java/org/jclouds/openstack/keystone/v2_0/handlers/RetryOnRenew.java
View File

@ -64,8 +64,8 @@ public class RetryOnRenew implements HttpRetryHandler {
retry = false; retry = false;
} else { } else {
byte[] content = closeClientButKeepContentStream(response); byte[] content = closeClientButKeepContentStream(response);
// Error when the session token expires: HTTP/1.1 401 Unauthorized, no body (or provider-specific response body) //TODO: what is the error when the session token expires??
if (content != null) { if (content != null && new String(content).contains("lease renew")) {
logger.debug("invalidating authentication token"); logger.debug("invalidating authentication token");
authenticationResponseCache.invalidateAll(); authenticationResponseCache.invalidateAll();
retry = true; retry = true;

2
apis/openstack-keystone/src/test/java/org/jclouds/openstack/keystone/v2_0/handlers/RetryOnRenewTest.java
View File

@ -53,7 +53,7 @@ public class RetryOnRenewTest {
cache.invalidateAll(); cache.invalidateAll();
expectLastCall(); expectLastCall();
expect(response.getPayload()).andReturn(Payloads.newStringPayload("")).anyTimes(); expect(response.getPayload()).andReturn(Payloads.newStringPayload("token expired, please renew")).anyTimes();
expect(response.getStatusCode()).andReturn(401).atLeastOnce(); expect(response.getStatusCode()).andReturn(401).atLeastOnce();
replay(command); replay(command);