Merge branch 'node-formidablegh-88-plattform-independent-upload-dir'

* node-formidablegh-88-plattform-independent-upload-dir:
  Document new uploadDir detection
  Plattform independent upload dir detection

Fixes node-formidable#88

Readme.md

@@ -142,9 +142,11 @@ Creates a new incoming form.
 
 The encoding to use for incoming form fields.
 
-#### incomingForm.uploadDir = '/tmp'
+#### incomingForm.uploadDir = process.env.TMP || '/tmp' || process.cwd()
 
-The directory for placing file uploads in. You can later on move them using `fs.rename()`.
+The directory for placing file uploads in. You can later on move them using
+`fs.rename()`. The default directoy is picked at module load time depending on
+the first existing directory from those listed above.
 
 #### incomingForm.keepExtensions = false
 

lib/incoming_form.js

@@ -1,5 +1,6 @@
 if (global.GENTLY) require = GENTLY.hijack(require);
 
+var fs = require('fs');
 var util = require('./util'),
     path = require('path'),
     File = require('./file'),
@@ -17,7 +18,7 @@ function IncomingForm() {
 
   this.maxFieldsSize = 2 * 1024 * 1024;
   this.keepExtensions = false;
-  this.uploadDir = '/tmp';
+  this.uploadDir = IncomingForm.UPLOAD_DIR;
   this.encoding = 'utf-8';
   this.headers = null;
   this.type = null;
@@ -32,6 +33,20 @@ function IncomingForm() {
 util.inherits(IncomingForm, EventEmitter);
 exports.IncomingForm = IncomingForm;
 
+IncomingForm.UPLOAD_DIR = (function() {
+  var dirs = [process.env.TMP, '/tmp', process.cwd()];
+  for (var i = 0; i < dirs.length; i++) {
+    var dir = dirs[i];
+    var isDirectory = false;
+
+    try {
+      isDirectory = fs.statSync(dir).isDirectory();
+    } catch (e) {}
+
+    if (isDirectory) return dir;
+  }
+})();
+
 IncomingForm.prototype.parse = function(req, cb) {
   this.pause = function() {
     try {