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(a)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(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org