Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

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

wants to merge 1 commit into from

Conversation

@reinhapa
Copy link

@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 <[email protected]>
@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