Skip to content

Add Jakarta REST Request and Response return support#828

Merged
arturobernalg merged 1 commit into
apache:masterfrom
arturobernalg:jakarta-rest-client
May 15, 2026
Merged

Add Jakarta REST Request and Response return support#828
arturobernalg merged 1 commit into
apache:masterfrom
arturobernalg:jakarta-rest-client

Conversation

@arturobernalg
Copy link
Copy Markdown
Member

@arturobernalg arturobernalg commented May 3, 2026

Support jakarta.ws.rs.core.Response as a resource method return type,
including access to response metadata and entity decoding. Responses
declared as Response are returned to the caller even for non-successful
status codes.

@arturobernalg arturobernalg requested a review from ok2c May 3, 2026 12:15
@ok2c
Copy link
Copy Markdown
Member

ok2c commented May 11, 2026

@arturobernalg Looks good overall. ClientRequest does not look very useful. I would drop it for now. I also would like to change the intermediate representation of the ClientResponse JSON content from byte array to JsonNode for better efficiency but this can be done as a separate change-set at a later point.

@arturobernalg
Copy link
Copy Markdown
Member Author

I also would like to change the intermediate representation of the ClientResponse JSON content from byte array to JsonNode

Done. I dropped ClientRequest / Request support and kept the change focused on Response.

I also changed JSON Response entity decoding to parse into JsonNode once and reuse that tree for subsequent readEntity(...) calls.

Copy link
Copy Markdown
Member

@ok2c ok2c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arturobernalg Please squash your changes

@arturobernalg arturobernalg force-pushed the jakarta-rest-client branch from 83f3e3e to c399de5 Compare May 14, 2026 07:31
@arturobernalg arturobernalg requested a review from ok2c May 14, 2026 07:31
Copy link
Copy Markdown
Member

@ok2c ok2c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arturobernalg Very good.

@arturobernalg
Copy link
Copy Markdown
Member Author

@arturobernalg Very good.

Thank you

@arturobernalg arturobernalg merged commit f275252 into apache:master May 15, 2026
10 checks passed
@arturobernalg
Copy link
Copy Markdown
Member Author

@arturobernalg Very good.

Thank you

@ok2c please let me know if you want to add any other feature to the Jackarta module

@ok2c
Copy link
Copy Markdown
Member

ok2c commented May 16, 2026

@arturobernalg Very good.

Thank you

@ok2c please let me know if you want to add any other feature to the Jackarta module

@arturobernalg I will start a thread on the dev list to discuss release plans.

We have plenty of new features already for the coming feature release. Please focus on polishing the new code and improving documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants