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