Skip to content

REST interface scanner code improvement#832

Merged
ok2c merged 1 commit into
apache:masterfrom
ok2c:jakarta-rest-code-improvements
May 24, 2026
Merged

REST interface scanner code improvement#832
ok2c merged 1 commit into
apache:masterfrom
ok2c:jakarta-rest-code-improvements

Conversation

@ok2c
Copy link
Copy Markdown
Member

@ok2c ok2c commented May 23, 2026

@arturobernalg I reworked some of your REST interface scanning code and made it use core parsing and formatting routines where available. Please review.

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

@arturobernalg I reworked some of your REST interface scanning code and made it use core parsing and formatting routines where available. Please review.

@ok2c
Looks good overall.

Should this use producesContentTypes instead? @produces maps to Accept; @consumes should only drive the request entity Content-Type.

@ok2c
Copy link
Copy Markdown
Member Author

ok2c commented May 24, 2026

@arturobernalg I suspect the current implementation of @Produces and @Consumes is wrong but this is a subject for a subsequent change-set. I wanted to make sure all your original tests still passed after the refactoring.

@ok2c ok2c merged commit 6dd7b5d into apache:master May 24, 2026
10 checks passed
@ok2c ok2c deleted the jakarta-rest-code-improvements branch May 24, 2026 10:20
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