Support for Jax-RS style "StreamingOutput" for JEX #674
Conversation
|
neat, thanks for taking the time to submit a pr |
you could try to write a test in the test-jex section. did that not work? |
I started to write a unit test, but the mocking was very difficult and it was quite dependent on statically initialized types. I just noticed the |
|
I will progress this further once this PR is resolved and new version of |
This commit only implements for the JEX server option.
Add a test # Conflicts: # tests/test-jex/src/main/resources/public/openapi.json
Head branch was pushed to by a user without write access
andponlin
force-pushed
the
andponlin-streaming-output-1
branch
from
725fc53 to
3586a17
Compare
That's done now. It has re-generated a number of OpenAPI specs, but it does this on |
http-generator-jex/src/main/java/io/avaje/http/generator/jex/ControllerMethodWriter.java
Outdated
Show resolved
Hide resolved
Swich back to the 3.3 of the avaje-jex and closing the `OutputStream` instead of flushing.
|
Hi @SentryMan / @rob-bygrave 👋 -- sorry to be a pain, but is it possible to get a release of it? |
|
let's get #679 merged first so we can do it all together |
3 participants
This PR only implements for the JEX server option; I have not done this for the other server options. I tried to write a unit test for this, but it is quite difficulty with the visibility of types and methods.