Change in ovirt-engine[master]: sercon: do not close streams in servlet
ofrenkel at redhat.com
ofrenkel at redhat.com
Mon Aug 3 14:40:17 UTC 2015
Omer Frenkel has submitted this change and it was merged.
Change subject: sercon: do not close streams in servlet
......................................................................
sercon: do not close streams in servlet
Do not close input/output explicitely in the servlet.
The Application Server is in charge of doing so.
This avoids double close and related stacktrace, which
leads in the good cause to scary noise in the logs,
or most likely prevents the servlet to work correctly.
Change-Id: I523cf1cb599a9f89dd91afed6245a348c01afb63
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/VMConsoleProxyServlet.java
1 file changed, 3 insertions(+), 6 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Francesco Romani: Verified
Omer Frenkel: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/43370
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I523cf1cb599a9f89dd91afed6245a348c01afb63
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the Engine-commits
mailing list