Skip to content

[REQ][Java][Microprofile] use WebApplicationException default runtime exception#21860

Open
reinhapa wants to merge 1 commit intoOpenAPITools:masterfrom
reinhapa:issue-21845-WebApplicationException
Open

[REQ][Java][Microprofile] use WebApplicationException default runtime exception#21860
reinhapa wants to merge 1 commit intoOpenAPITools:masterfrom
reinhapa:issue-21845-WebApplicationException

Conversation

@reinhapa
Copy link
Copy Markdown

@reinhapa reinhapa commented Sep 1, 2025 •
edited
Loading

Implements the proposed #21845 behavior.

PR checklist

  • Read the contribution guidelines.
  • Pull Request title clearly describes the work in the pull request and Pull Request description provides details about how to validate the work. Missing information here may result in delayed response from the community.
  • Run the following to build the project and update samples: 
    ./mvnw clean package || exit
./bin/generate-samples.sh ./bin/configs/*.yaml || exit
./bin/utils/export_docs_generators.sh || exit

@reinhapa reinhapa changed the title First attempt changing #21845 behavior Use WebApplicationException as default Microprofile API exception Sep 2, 2025
@reinhapa reinhapa changed the title Use WebApplicationException as default Microprofile API exception [REQ][Java][Microprofile] use WebApplicationException default runtime exception Sep 2, 2025
@reinhapa reinhapa force-pushed the issue-21845-WebApplicationException branch 4 times, most recently from 4f11e9a to 2dd4ab5 Compare September 2, 2025 18:42
@reinhapa reinhapa marked this pull request as ready for review September 2, 2025 18:57
@reinhapa reinhapa force-pushed the issue-21845-WebApplicationException branch 6 times, most recently from b434987 to dc19465 Compare November 4, 2025 16:38
@reinhapa
Implements OpenAPITools#21845 behavior
0013874 
- Not generating 'ApiExceptionMapper' if
  'microProfileRegisterExceptionMapper' is set to 'false'
- Only import RegisterRestClient if actually needed
- Default 'useRuntimeException' to 'true' for Microprofile and use
  'WebApplicationException' instead of custom 'ApiException'

Signed-off-by: Patrick Reinhart <patrick@reini.net>
@reinhapa reinhapa force-pushed the issue-21845-WebApplicationException branch from dc19465 to 0013874 Compare November 5, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@reinhapa