Skip to content
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.

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

NPE on "204 - no content"-response with OpenAPI generated Jersey client #44

Closed
GTaenzer opened this issue Jan 12, 2023 · 1 comment
Closed
Labels
bug Something isn't working

Comments

@GTaenzer
Copy link
Collaborator

GTaenzer commented Jan 12, 2023

Affected version
4.4.0

Actual behavior
Generated deleteReservation request in ReservationServiceHandler fails with NPE on 204:
Exception during FilterChain execution
java.lang.NullPointerException: Cannot invoke "org.glassfish.grizzly.http.HttpRequestPacket.getProcessingState()" because the return value of "org.glassfish.grizzly.http.HttpClientFilter$ClientHttpResponseImpl.getRequest()" is null
at org.glassfish.grizzly.http.HttpClientFilter$ClientHttpResponseImpl.getProcessingState(HttpClientFilter.java:644)

Expected behavior
empty body should be ok for 204 responses

To Reproduce
Steps to reproduce the behavior:

  1. create a reservation
  2. try to delete the reservation using deleteReservation request in ReservationServiceHandler

AB#82534

@GTaenzer GTaenzer added the bug Something isn't working label Jan 12, 2023
GTaenzer added a commit that referenced this issue Jan 12, 2023
Philipp-Borchert-ISH pushed a commit that referenced this issue Jan 20, 2023
…#41) (#42)

* starting work on a reservation service (#41)

* registered reservation service handler (#41)

* building a reservation service handler (#41)

* building an inventory service handler (#41)

* combined inventory and reservation service handler (#41)

* undo new version (#41)

* delete reservation added in reservation service handler (#41)

* chore: update jersey to 3.1.0 (#44)

* feat: yaml for reservation API (#41)

* feat: comma separated list as utility for inventory API (#41)

* chore: next version (#41)

* feat: version independent enum mapping (#41)

* API-Break: renamed enum values in RESERVATION_STATE (#41)

* feat: version independent enum mapping - fix import (#41)

* feat: undo API-Break (#41)

* fix: removed yaml json schema (#41)

* reviewed (#41)

* fixing API break (#41)

* reviewed (#41)

* don't ignore lifetime (#41)

* using JAVA 8 times (#41)
@jpgorzny
Copy link
Contributor

duplicate of #91

@jpgorzny jpgorzny closed this as not planned Won't fix, can't repro, duplicate, stale Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants