mirror of
https://github.com/OpenAPITools/openapi-generator.git
synced 2025-05-12 20:50:55 +00:00
fix connection leak on retrofit OAuth token renewal (#7750)
This commit is contained in:
parent
0adbf7e51f
commit
53eeb0c049
@ -113,8 +113,14 @@ public class OAuth implements Interceptor {
|
|||||||
|
|
||||||
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
||||||
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
||||||
if (updateAccessToken(requestAccessToken)) {
|
try {
|
||||||
return retryingIntercept( chain, false );
|
if (updateAccessToken(requestAccessToken)) {
|
||||||
|
response.body().close();
|
||||||
|
return retryingIntercept( chain, false );
|
||||||
|
}
|
||||||
|
} catch (Exception e) {
|
||||||
|
response.body().close();
|
||||||
|
throw e;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return response;
|
return response;
|
||||||
|
@ -113,8 +113,14 @@ public class OAuth implements Interceptor {
|
|||||||
|
|
||||||
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
||||||
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
||||||
if (updateAccessToken(requestAccessToken)) {
|
try {
|
||||||
return retryingIntercept( chain, false );
|
if (updateAccessToken(requestAccessToken)) {
|
||||||
|
response.body().close();
|
||||||
|
return retryingIntercept( chain, false );
|
||||||
|
}
|
||||||
|
} catch (Exception e) {
|
||||||
|
response.body().close();
|
||||||
|
throw e;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return response;
|
return response;
|
||||||
|
@ -113,8 +113,14 @@ public class OAuth implements Interceptor {
|
|||||||
|
|
||||||
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
||||||
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
||||||
if (updateAccessToken(requestAccessToken)) {
|
try {
|
||||||
return retryingIntercept( chain, false );
|
if (updateAccessToken(requestAccessToken)) {
|
||||||
|
response.body().close();
|
||||||
|
return retryingIntercept( chain, false );
|
||||||
|
}
|
||||||
|
} catch (Exception e) {
|
||||||
|
response.body().close();
|
||||||
|
throw e;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return response;
|
return response;
|
||||||
|
@ -113,8 +113,14 @@ public class OAuth implements Interceptor {
|
|||||||
|
|
||||||
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
||||||
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
||||||
if (updateAccessToken(requestAccessToken)) {
|
try {
|
||||||
return retryingIntercept( chain, false );
|
if (updateAccessToken(requestAccessToken)) {
|
||||||
|
response.body().close();
|
||||||
|
return retryingIntercept( chain, false );
|
||||||
|
}
|
||||||
|
} catch (Exception e) {
|
||||||
|
response.body().close();
|
||||||
|
throw e;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return response;
|
return response;
|
||||||
|
@ -113,8 +113,14 @@ public class OAuth implements Interceptor {
|
|||||||
|
|
||||||
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
||||||
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
||||||
if (updateAccessToken(requestAccessToken)) {
|
try {
|
||||||
return retryingIntercept( chain, false );
|
if (updateAccessToken(requestAccessToken)) {
|
||||||
|
response.body().close();
|
||||||
|
return retryingIntercept( chain, false );
|
||||||
|
}
|
||||||
|
} catch (Exception e) {
|
||||||
|
response.body().close();
|
||||||
|
throw e;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return response;
|
return response;
|
||||||
|
@ -113,8 +113,14 @@ public class OAuth implements Interceptor {
|
|||||||
|
|
||||||
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
// 401/403 most likely indicates that access token has expired. Unless it happens two times in a row.
|
||||||
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
if ( response != null && (response.code() == HTTP_UNAUTHORIZED || response.code() == HTTP_FORBIDDEN) && updateTokenAndRetryOnAuthorizationFailure ) {
|
||||||
if (updateAccessToken(requestAccessToken)) {
|
try {
|
||||||
return retryingIntercept( chain, false );
|
if (updateAccessToken(requestAccessToken)) {
|
||||||
|
response.body().close();
|
||||||
|
return retryingIntercept( chain, false );
|
||||||
|
}
|
||||||
|
} catch (Exception e) {
|
||||||
|
response.body().close();
|
||||||
|
throw e;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return response;
|
return response;
|
||||||
|
Loading…
x
Reference in New Issue
Block a user