CIBAPingCallbackService is unable to read the response of the callback

By: Lukasz Golinski named 24 Jan 2022 at 9:34 a.m. CST

5 Responses
Lukasz Golinski gravatar
## Expected behavior During the Ciba ping callback request oxauth should be able to parse the response of the callback and successfully log the response in the following line of code. https://github.com/GluuFederation/oxAuth/blob/749e52ad0c6d11969bd546acc5f545acfc8f5bef/Server/src/main/java/org/gluu/oxauth/ciba/CIBAPingCallbackService.java#L42 ## Actual behavior During the Ciba ping callback response is successful. But during mapping of the response to the entity that is happening both in the constructor of the super class BaseResponse and in the extended class PingCallbackResponse, IllegalStateException is thrown with the following message: *RESTEASY003765: Response is closed.* ## Minimized example 1. PING rest call happening here: https://github.com/GluuFederation/oxAuth/blob/b3f2f92a6543c9d0bac683944f5bf13368ef9c06/Client/src/main/java/org/gluu/oxauth/client/ciba/ping/PingCallbackClient.java#L64 2. The call is successful. 3. The entity of the response is being read for the first time here in the constructor of the BaseResponse class of: https://github.com/GluuFederation/oxAuth/blob/0b7239c0756014fcd136d0ce5601452cc83caa5a/Client/src/main/java/org/gluu/oxauth/client/BaseResponse.java#L38 4. During the second call to readEntity inside the PingCallbackResponse class: https://github.com/GluuFederation/oxAuth/blob/0b7239c0756014fcd136d0ce5601452cc83caa5a/Client/src/main/java/org/gluu/oxauth/client/ciba/ping/PingCallbackResponse.java#L25 IllegalStateException is thrown with the following message RESTEASY003765: Response is closed. ## Log files You can find it in the attached file.
Gluu 4.3 Docker 20.1
closed

Answers

By Milton Ch. staff 25 Jan 2022 at 6:21 a.m. CST

Copy
Milton Ch. gravatar
Hi Lukasz, I have identified root reason, there was a RESTEasy library update and beside it, some lines were updated, looks like solution should not be complex, but let me coordinate it internally. Thanks for reporting that finding.

By Milton Ch. staff 27 Jan 2022 at 8:03 a.m. CST

Copy
Milton Ch. gravatar
Hi Lukasz, let you know that the fix just got merged into master branch, is that ok for you? should we release that change in one specific version?

By Lukasz Golinski named 28 Jan 2022 at 2:35 a.m. CST

Copy
Lukasz Golinski gravatar
Hi Milton, Thank you for the update. Currently we are using following oxauth image: gluufederation/oxauth:4.3.1_01 If this fix could show up in the next version of this image, that would be helpful.

By Milton Ch. staff 01 Feb 2022 at 12:08 p.m. CST

Copy
Milton Ch. gravatar
Hey Lukasz, fix has been released as part of package `4.3.1.sp1`, it's based on version 4.3.1, but it contains the fix for this CIBA stuff too.

By Milton Ch. staff 03 Feb 2022 at 8:03 p.m. CST

Copy
Milton Ch. gravatar
Closing this one since fix was implemented accordingly. Feel free to re-open whether there is something missing.

Post an answer