[TOMEE-1862] TomcatWebAppBuilder: no need to track contextXml inputstream

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 7.0.2
• Assignee: Romain Manni-Bucau
• Reporter: Romain Manni-Bucau
• Created: 08/Jul/16 11:33
• Updated: 08/Jul/16 11:34
• Resolved: 08/Jul/16 11:34

Description

From the list (thanks Svetlin Zarev):

Hi,

TomcatWebappDeployer::deployWebApps() does not close the InputStream to the
context.xml it opens.It opens the stream on line 462 or 471 (branch 7.0.1)
, an uses it only to check if it's null on line 500 without ever reading
it.

Leaving the stream open may cause application undeployment issues, because
the users will not be able to delete the WAR/or the context.xml from the FS
on operation systems like windows that lock the file if there are open
handles to it.

Kind regards,
Svetlin