Skip to content

Conversation

cmoine-swi
Copy link

@cmoine-swi cmoine-swi commented Sep 3, 2025

PR for #21424

  • Add generator option useJBossResponse to use typed org.jboss.resteasy.reactive.RestResponse instead of javax.ws.rs.core.Response (see here)
  • Add tests
  • Update documentation

@cmoine-swi cmoine-swi marked this pull request as draft September 3, 2025 09:32
@wing328
Copy link
Member

wing328 commented Sep 4, 2025

thanks for the PR

cc @bbdouglas (2017/07) @sreeshas (2017/08) @jfiala (2017/08) @lukoyanov (2017/09) @cbornet (2017/09) @jeff9finger (2018/01) @karismann (2019/03) @Zomzog (2019/04) @lwlee2608 (2019/10) @martin-mfg (2023/08)

@wing328
Copy link
Member

wing328 commented Sep 4, 2025

did you test this PR locally to confirm it works for your use cases?

@cmoine-swi
Copy link
Author

Hi @wing328, I just tested, and I confirm that version "7.16.0-SNAPSHOT" works on my project 👍

@wing328
Copy link
Member

wing328 commented Sep 6, 2025

i tested with

java -jar modules/openapi-generator-cli/target/openapi-generator-cli.jar generate -g jaxrs-spec --library quarkus -i modules/openapi-generator/src/test/resources/3_0/petstore.yaml -o /tmp/jaxrs2/ --additional-properties returnJBossResponse=true,interfaceOnly=true

but the output won't compile

does it work for you locally in your machine?

@wing328
Copy link
Member

wing328 commented Sep 6, 2025

please also follow step 3 in the PR checklist to update samples and doc

@wing328
Copy link
Member

wing328 commented Sep 6, 2025

instead of adding a new option, what about updating the existing one returnResponse to have one additional option to support your use cases?

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