Update OAuth2AccessTokenJackson2Deserializer #1173
Conversation
Some services use expires_in: 0 to say token never expire
@alatushkin Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
@alatushkin Thank you for signing the Contributor License Agreement! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @alatushkin. Can you please add a test and we'll get this merged.
@@ -92,7 +92,7 @@ else if (OAuth2AccessToken.SCOPE.equals(name)) { | |||
|
|||
DefaultOAuth2AccessToken accessToken = new DefaultOAuth2AccessToken(tokenValue); | |||
accessToken.setTokenType(tokenType); | |||
if (expiresIn != null) { | |||
if (expiresIn != null && expiresIn > 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please add a test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi.
Sure. I'l try to do this in next three days.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closing in favour of #1862 |
Some services use expires_in: 0 to say token never expire
#1172