[UWS] Replace the public static variable for the tmp directory by a class

attribute.

In this way, it is possible to run two different instances of a UWS/TAP service
with a different temporary directory in the same JVM.

src/uws/service/file/LocalUWSFileManager.java

@@ -16,7 +16,7 @@ package uws.service.file;
  * You should have received a copy of the GNU Lesser General Public License
  * along with UWSLibrary.  If not, see <http://www.gnu.org/licenses/>.
  *
- * Copyright 2012-2017 - UDS/Centre de Données astronomiques de Strasbourg (CDS),
+ * Copyright 2012-2018 - UDS/Centre de Données astronomiques de Strasbourg (CDS),
  *                       Astronomisches Rechen Institut (ARI)
  */
 
@@ -53,26 +53,31 @@ import uws.service.log.UWSLog.LogLevel;
 import uws.service.request.UploadFile;
 
 /**
- * <p>All UWS files are stored in the local machine into the specified directory.</p>
+ * All UWS files are stored in the local machine into the specified directory.
  *
  * <p>
- * 	The name of the log file, the result files and the backup files may be customized by overriding the following functions:
- * 	{@link #getLogFileName(uws.service.log.UWSLog.LogLevel, String)}, {@link #getResultFileName(Result, UWSJob)}, {@link #getBackupFileName(JobOwner)} and {@link #getBackupFileName()}.
+ * 	The name of the log file, the result files and the backup files may be
+ * 	customised by overriding the following functions:
+ * 	{@link #getLogFileName(uws.service.log.UWSLog.LogLevel, String)},
+ * 	{@link #getResultFileName(Result, UWSJob)},
+ * 	{@link #getBackupFileName(JobOwner)} and {@link #getBackupFileName()}.
  * </p>
  *
  * <p>
- * 	By default, results and backups are grouped by owner/user and owners/users are grouped thanks to {@link DefaultOwnerGroupIdentifier}.
- * 	By using the appropriate constructor, you can change these default behaviors.
+ * 	By default, results and backups are grouped by owner/user and owners/users
+ * 	are grouped thanks to {@link DefaultOwnerGroupIdentifier}. By using the
+ * 	appropriate constructor, you can change these default behaviours.
  * </p>
  *
  * <p>
- * 	A log file rotation is set by default so that avoiding a too big log file after several months/years of use.
- * 	By default the rotation is done every month on the 1st at 6am. This frequency can be changed easily thanks to the function
- * 	{@link #setLogRotationFreq(String)}.
+ * 	A log file rotation is set by default so that avoiding a too big log file
+ * 	after several months/years of use. By default the rotation is done every
+ * 	month on the 1st at 6am. This frequency can be changed easily thanks to the
+ * 	function {@link #setLogRotationFreq(String)}.
  * </p>
  *
  * @author Gr&eacute;gory Mantelet (CDS;ARI)
- * @version 4.4 (07/2018)
+ * @version 4.4 (08/2018)
  */
 public class LocalUWSFileManager implements UWSFileManager {
 
@@ -87,6 +92,10 @@ public class LocalUWSFileManager implements UWSFileManager {
 	/** Directory in which all files managed by this class will be written and read. */
 	protected final File rootDirectory;
 
+	/** Directory in which temporary files (e.g. uploads) should be stored.
+	 * @since 4.4 */
+	protected File tmpDirectory = new File(System.getProperty("java.io.tmpdir"));
+
 	/** Output toward the service log file. */
 	protected PrintWriter logOutput = null;
 	/** Frequency at which the log file must be "rotated" (the file is renamed with the date of its first write and a new log file is created).
@@ -439,6 +448,21 @@ public class LocalUWSFileManager implements UWSFileManager {
 			return new File(upload.getLocation());
 	}
 
+	@Override
+	public File getTmpDirectory(){
+		return tmpDirectory;
+	}
+
+	@Override
+	public boolean setTmpDirectory(final File newTmpDir){
+		if (newTmpDir == null || !newTmpDir.exists() || !newTmpDir.isDirectory() || !newTmpDir.canRead() || !newTmpDir.canWrite())
+			return false;
+		else{
+			tmpDirectory = newTmpDir;
+			return true;
+		}
+	}
+
 	@Override
 	public InputStream getUploadInput(final UploadFile upload) throws IOException{
 		// Check the source file:

src/uws/service/request/MultipartParser.java

@@ -16,7 +16,8 @@ package uws.service.request;
  * You should have received a copy of the GNU Lesser General Public License
  * along with UWSLibrary.  If not, see <http://www.gnu.org/licenses/>.
  *
- * Copyright 2014 - Astronomisches Rechen Institut (ARI)
+ * Copyright 2014-2018 - UDS/Centre de Données astronomiques de Strasbourg (CDS),
+ *                       Astronomisches Rechen Institut (ARI)
  */
 
 import java.io.File;
@@ -28,36 +29,40 @@ import java.util.Map;
 
 import javax.servlet.http.HttpServletRequest;
 
+import com.oreilly.servlet.MultipartRequest;
+import com.oreilly.servlet.multipart.FileRenamePolicy;
+
 import uws.UWSException;
 import uws.service.UWS;
 import uws.service.file.UWSFileManager;
 
-import com.oreilly.servlet.MultipartRequest;
-import com.oreilly.servlet.multipart.FileRenamePolicy;
-
 /**
- * <p>Extract parameters encoded using the Content-type multipart/form-data
- * in an {@link HttpServletRequest}.</p>
+ * Extract parameters encoded using the Content-type multipart/form-data
+ * in an {@link HttpServletRequest}.
  *
  * <p>
- * 	The created file(s) is(are) stored in the temporary upload directory ({@link UWSFileManager#TMP_UPLOAD_DIR} ; this attribute can be modified if needed).
- * 	This directory is supposed to be emptied regularly in case it is forgotten at any moment by the UWS service implementation to delete unused request files.
+ * 	The created file(s) is(are) stored in the temporary upload directory
+ * 	({@link UWSFileManager#getTmpDirectory()}. This directory is supposed to be
+ * 	emptied regularly in case it is forgotten at any moment by the service
+ * 	implementation to delete unused request files.
  * </p>
  *
  * <p>
- * 	The size of the full request body is limited by the static attribute {@link #SIZE_LIMIT} before the creation of the file.
- * 	Its default value is: {@link #DEFAULT_SIZE_LIMIT}={@value #DEFAULT_SIZE_LIMIT} bytes.
+ * 	The size of the full request body is limited by the static attribute
+ * 	{@link #SIZE_LIMIT} before the creation of the file. Its default value is:
+ * 	{@link #DEFAULT_SIZE_LIMIT}={@value #DEFAULT_SIZE_LIMIT} bytes.
  * </p>
  *
  * <p>
- * 	By default, this {@link RequestParser} overwrite parameter occurrences in the map: that's to say if a parameter is provided several times,
- * 	only the last value will be kept. This behavior can be changed by overwriting the function {@link #consumeParameter(String, Object, Map)}
- * 	of this class.
+ * 	By default, this {@link RequestParser} overwrite parameter occurrences in
+ * 	the map: that's to say if a parameter is provided several times, only the
+ * 	last value will be kept. This behaviour can be changed by overwriting the
+ * 	function {@link #consumeParameter(String, Object, Map)} of this class.
  * </p>
  *
- * @author Gr&eacute;gory Mantelet (ARI)
- * @version 4.1 (12/2014)
- * @since 4.1
+ * @author Gr&eacute;gory Mantelet (ARI;CDS)
+ * @version 4.4 (08/2018)
+ * @since 4.4
  */
 public class MultipartParser implements RequestParser {
 
@@ -135,7 +140,7 @@ public class MultipartParser implements RequestParser {
 		try{
 
 			// Parse the request body:
-			multipart = new MultipartRequest(request, UWSFileManager.TMP_UPLOAD_DIR.getPath(), (SIZE_LIMIT < 0 ? DEFAULT_SIZE_LIMIT : SIZE_LIMIT), new FileRenamePolicy(){
+			multipart = new MultipartRequest(request, fileManager.getTmpDirectory().getPath(), (SIZE_LIMIT < 0 ? DEFAULT_SIZE_LIMIT : SIZE_LIMIT), new FileRenamePolicy() {
 				@Override
 				public File rename(File file){
 					Object reqID = request.getAttribute(UWS.REQ_ATTRIBUTE_ID);
@@ -188,7 +193,7 @@ public class MultipartParser implements RequestParser {
 			throw new UWSException(UWSException.INTERNAL_SERVER_ERROR, ioe, "Internal Error => Impossible to extract parameters from the Multipart HTTP request!");
 		}catch(IllegalArgumentException iae){
 			String confError = iae.getMessage();
-			if (UWSFileManager.TMP_UPLOAD_DIR == null)
+			if (fileManager.getTmpDirectory() == null)
 				confError = "Missing upload directory!";
 			throw new UWSException(UWSException.INTERNAL_SERVER_ERROR, iae, "Internal Error: Incorrect UPLOAD configuration: " + confError);
 		}
@@ -217,7 +222,8 @@ public class MultipartParser implements RequestParser {
 		if (allParams.containsKey(name) && allParams.get(name) instanceof UploadFile){
 			try{
 				((UploadFile)allParams.get(name)).deleteFile();
-			}catch(IOException ioe){}
+			}catch(IOException ioe){
+			}
 		}
 
 		// Put the given value in the given map:
@@ -225,16 +231,18 @@ public class MultipartParser implements RequestParser {
 	}
 
 	/**
-	 * <p>Utility method that determines whether the content of the given request is a multipart/form-data.</p>
+	 * Utility method that determines whether the content of the given request
+	 * is a multipart/form-data.
 	 *
 	 * <p><i>Important:
-	 * 	This function just test the content-type of the request. The HTTP method (e.g. GET, POST, ...) is not tested.
+	 * 	This function just test the content-type of the request. The HTTP method
+	 * 	(e.g. GET, POST, ...) is not tested.
 	 * </i></p>
 	 *
 	 * @param request The servlet request to be evaluated. Must be non-null.
 	 *
-	 * @return	<i>true</i> if the request is multipart,
-	 *        	<i>false</i> otherwise.
+	 * @return	<code>true</code> if the request is multipart,
+	 *        	<code>false</code> otherwise.
 	 */
 	public static final boolean isMultipartContent(final HttpServletRequest request){
 		// Extract the content type and determine if it is a multipart request (its content type should start by multipart/form-data"):

src/uws/service/request/NoEncodingParser.java

@@ -37,21 +37,24 @@ import uws.service.UWS;
 import uws.service.file.UWSFileManager;
 
 /**
- * <p>This parser merely copies the whole HTTP request content inside a file.
- * It names this file: "JDL" (Job Description Language).</p>
+ * This parser merely copies the whole HTTP request content inside a file.
+ * It names this file: "JDL" (Job Description Language).
  *
  * <p>
- * 	The created file is stored in the temporary upload directory ({@link UWSFileManager#TMP_UPLOAD_DIR} ; this attribute can be modified if needed).
- * 	This directory is supposed to be emptied regularly in case it is forgotten at any moment by the UWS service implementation to delete unused request files.
+ * 	The created file is stored in the temporary upload directory
+ * 	({@link UWSFileManager#getTmpDirectory()}. This directory is supposed to be
+ * 	emptied regularly in case it is forgotten at any moment by the service
+ * 	implementation to delete unused request files.
  * </p>
  *
  * <p>
- * 	The size of the JDL is limited by the static attribute {@link #SIZE_LIMIT} before the creation of the file.
- * 	Its default value is: {@link #DEFAULT_SIZE_LIMIT}={@value #DEFAULT_SIZE_LIMIT} bytes.
+ * 	The size of the JDL is limited by the static attribute {@link #SIZE_LIMIT}
+ * 	before the creation of the file. Its default value is:
+ * 	{@link #DEFAULT_SIZE_LIMIT}={@value #DEFAULT_SIZE_LIMIT} bytes.
  * </p>
  *
- * @author Gr&eacute;gory Mantelet (ARI)
- * @version 4.1 (11/2014)
+ * @author Gr&eacute;gory Mantelet (CDS;ARI)
+ * @version 4.4 (08/2018)
  * @since 4.1
  */
 public class NoEncodingParser implements RequestParser {
@@ -106,7 +109,7 @@ public class NoEncodingParser implements RequestParser {
 		Object reqID = request.getAttribute(UWS.REQ_ATTRIBUTE_ID);
 		if (reqID == null || !(reqID instanceof String))
 			reqID = (new Date()).getTime();
-		File f = new File(UWSFileManager.TMP_UPLOAD_DIR, "REQUESTBODY_" + reqID);
+		File f = new File(fileManager.getTmpDirectory(), "REQUESTBODY_" + reqID);
 		OutputStream output = null;
 		InputStream input = null;
 		long totalLength = 0;
@@ -141,12 +144,14 @@ public class NoEncodingParser implements RequestParser {
 			if (input != null){
 				try{
 					input.close();
-				}catch(IOException ioe2){}
+				}catch(IOException ioe2){
+				}
 			}
 			if (output != null){
 				try{
 					output.close();
-				}catch(IOException ioe2){}
+				}catch(IOException ioe2){
+				}
 			}
 		}
 

src/uws/service/request/XMLRequestParser.java

@@ -169,7 +169,7 @@ public class XMLRequestParser implements RequestParser {
 		Object reqID = request.getAttribute(UWS.REQ_ATTRIBUTE_ID);
 		if (reqID == null || !(reqID instanceof String))
 			reqID = (new Date()).getTime();
-		File xmlFile = new File(UWSFileManager.TMP_UPLOAD_DIR, "JOB_DESCRIPTION_" + reqID);
+		File xmlFile = new File(fileManager.getTmpDirectory(), "JOB_DESCRIPTION_" + reqID);
 
 		OutputStream output = null;
 		InputStream input = null;
@@ -254,12 +254,14 @@ public class XMLRequestParser implements RequestParser {
 			if (output != null){
 				try{
 					output.close();
-				}catch(IOException ioe2){}
+				}catch(IOException ioe2){
+				}
 			}
 			if (input != null){
 				try{
 					input.close();
-				}catch(IOException ioe2){}
+				}catch(IOException ioe2){
+				}
 			}
 		}
 
@@ -312,7 +314,8 @@ public class XMLRequestParser implements RequestParser {
 			if (input != null){
 				try{
 					input.close();
-				}catch(IOException ioe){}
+				}catch(IOException ioe){
+				}
 			}
 		}
 	}