Merge pull request #39 from edeustace/close-streams

close streams

Author: edeustace

plugin/app/com/ee/assets/transformers/ByteArrayWriter.scala

@@ -19,7 +19,9 @@ class ByteArrayWriter(fileFn: String => File) extends Transformer[Array[Byte], U
       val fileOut = fileFn(e.path)
       logger.trace(s"from ${e.path} write bytes to ${fileOut.getPath}")
       fileOut.getParentFile.mkdirs()
-      IOUtils.write(e.contents, new FileOutputStream(fileOut))
+      val os = new FileOutputStream(fileOut)
+      IOUtils.write(e.contents, os)
+      os.close()
       PathElement(e.path)
     }
   }

plugin/app/com/ee/assets/transformers/PlayResourceReader.scala

@@ -23,7 +23,16 @@ class PlayResourceReader extends Transformer[Unit, String] {
 
         val contents = url.map {
           u =>
-            IOUtils.toString(u.openStream())
+            val stream = u.openStream()
+            try {
+              val out = IOUtils.toString(stream)
+              stream.close()
+              out
+            } catch {
+              case e : Throwable => throw new AssetsLoaderException(s"Error with stream ${u}", e) 
+            } finally{
+              IOUtils.closeQuietly(stream)
+            }
         }.orElse {
           val errorMsg = s"can't load path: ${e.path}"
           throw new AssetsLoaderException(errorMsg)