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

[3.15] Always close OutputStream in RESTEasy Classic #46426

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Feb 21, 2025

Fixes: #46412

Copy link

quarkus-bot bot commented Feb 21, 2025

/cc @aloubyansky (3.15), @gastaldi (3.15), @jmartisk (3.15), @rsvoboda (3.15)

@quarkus-bot quarkus-bot bot changed the title Always close OutputStream in RESTEasy Classic [3.15] Always close OutputStream in RESTEasy Classic Feb 21, 2025
@gsmet gsmet requested a review from geoand February 21, 2025 16:03
@gsmet gsmet changed the title [3.15] Always close OutputStream in RESTEasy Classic WIP - [3.15] Always close OutputStream in RESTEasy Classic Feb 21, 2025
@gsmet
Copy link
Member Author

gsmet commented Feb 21, 2025

Let's not merge them right away. I will merge them next week.

Copy link

quarkus-bot bot commented Feb 21, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit c667a2a.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@gastaldi
Copy link
Contributor

@gsmet I think this can be merged now?

@gsmet
Copy link
Member Author

gsmet commented Feb 24, 2025

Yes. Don’t worry, it’s under control, I’ll do it tomorrow morning :).

@gsmet gsmet changed the title WIP - [3.15] Always close OutputStream in RESTEasy Classic [3.15] Always close OutputStream in RESTEasy Classic Feb 25, 2025
@gsmet gsmet merged commit 70ffbd0 into quarkusio:3.15 Feb 25, 2025
52 checks passed
@gsmet gsmet added this to the 3.15.4 milestone Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants