7:09 p.m.
Author: ips
Date: 2011-07-29 13:09:55 -0400 (Fri, 29 Jul 2011)
New Revision: 1025
Modified:
trunk/core/src/main/java/org/jboss/on/embedded/ui/content/AbstractFileUploadAction.java
trunk/core/src/main/webapp/WEB-INF/components.xml
Log:
[EMBJOPR-259] fix so uploaded EAR and WAR files are not stored in byte arrays to prevent
potentially consuming a lot of the app server's heap
(https://issues.jboss.org/browse/EMBJOPR-359)
Modified:
trunk/core/src/main/java/org/jboss/on/embedded/ui/content/AbstractFileUploadAction.java
===================================================================
---
trunk/core/src/main/java/org/jboss/on/embedded/ui/content/AbstractFileUploadAction.java 2011-07-28
21:24:15 UTC (rev 1024)
+++
trunk/core/src/main/java/org/jboss/on/embedded/ui/content/AbstractFileUploadAction.java 2011-07-29
17:09:55 UTC (rev 1025)
@@ -1,6 +1,6 @@
/*
* Embedded Jopr Project
- * Copyright (C) 2006-2009 Red Hat, Inc.
+ * Copyright (C) 2006-2011 Red Hat, Inc.
* All rights reserved.
*
* This program is free software; you can redistribute it and/or modify
@@ -21,6 +21,7 @@
import java.io.File;
import java.io.FileOutputStream;
+import java.io.InputStream;
import java.io.IOException;
import javax.faces.application.FacesMessage;
@@ -31,6 +32,8 @@
import org.jboss.seam.annotations.In;
import org.jboss.seam.faces.FacesMessages;
+import org.rhq.core.util.stream.StreamUtil;
+
import org.jboss.on.embedded.ui.BootstrapAction;
import org.jboss.on.embedded.ui.NavigationAction;
@@ -47,7 +50,7 @@
private final Log log = LogFactory.getLog(this.getClass());
private String fileName;
- private byte[] file;
+ private InputStream file;
private String fileContentType;
/**
@@ -72,14 +75,13 @@
{
uploadsDir.mkdirs();
- String baseName = new File(this.fileName).getName();
+ String baseName = new File(this.fileName).getName();
tempFile = new File(uploadsDir, baseName);
log.debug("Writing uploaded file to '" + tempFile +
"'...");
FileOutputStream fos = new FileOutputStream(tempFile);
- fos.write(this.file);
- log.debug("Wrote " + this.file.length + " bytes to '"
+ tempFile + "'.");
- fos.close();
+ StreamUtil.copy(this.file, fos, true);
+ log.debug("Wrote " + tempFile.length() + " bytes to
'" + tempFile + "'.");
}
catch (IOException e)
{
@@ -101,12 +103,12 @@
this.fileName = fileName;
}
- public byte[] getFile()
+ public InputStream getFile()
{
return file;
}
- public void setFile(byte[] file)
+ public void setFile(InputStream file)
{
this.file = file;
}
@@ -121,3 +123,4 @@
this.fileContentType = fileContentType;
}
}
+
Modified: trunk/core/src/main/webapp/WEB-INF/components.xml
===================================================================
--- trunk/core/src/main/webapp/WEB-INF/components.xml 2011-07-28 21:24:15 UTC (rev 1024)
+++ trunk/core/src/main/webapp/WEB-INF/components.xml 2011-07-29 17:09:55 UTC (rev 1025)
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
~ Embedded Jopr Project
- ~ Copyright (C) 2006-2009 Red Hat, Inc.
+ ~ Copyright (C) 2006-2011 Red Hat, Inc.
~ All rights reserved.
~
~ This program is free software; you can redistribute it and/or modify
@@ -21,8 +21,9 @@
<components xmlns="http://jboss.com/products/seam/components"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns:security="http://jboss.com/products/seam/security"
+ xmlns:web="http://jboss.com/products/seam/web"
xsi:schemaLocation=
- "http://jboss.com/products/seam/components
http://jboss.com/products/seam/components-2.0.xsd
+ "http://jboss.com/products/seam/components
http://jboss.com/products/seam/components-2.0.xsd
http://jboss.com/products/seam/security
http://jboss.com/products/seam/security-2.0.xsd">
<component name="org.jboss.seam.core.init">
@@ -39,6 +40,10 @@
<property name="scanPeriod">60000</property>
</component>
+ <!-- write uploaded WAR or EAR files out to temp files rather than storing
+ them in memory, to avoid consuming a lot of the app server's heap -->
+ <web:multipart-filter create-temp-files="true"/>
+
<!-- security stuff -->
<security:identity authenticate-method="#{authenticator.authenticate}"
jaas-config-name="jmx-console"/>
@@ -53,3 +58,4 @@
</event>
</components>
+