Merge pull request danwrong#109 from max-koehler/HTTP-Location-Fix

Fixed redirect handling for HTTP status code 303

Author: danwrong

.gitignore

@@ -1,3 +1,4 @@
 *.swp
 .[Dd][Ss]_store
 node_modules
+.idea

lib/restler.js

@@ -119,9 +119,18 @@ mixin(Request.prototype, {
 
     if (self._isRedirect(response) && self.options.followRedirects) {
       try {
-        self.url = url.parse(url.resolve(self.url.href, response.headers['location']));
-        self._retry();
-        // todo handle somehow infinite redirects
+        // 303 should redirect and retrieve content with the GET method
+        // http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html#sec10.3.4
+        if (response.statusCode === 303) {
+            self.url    = url.parse(url.resolve(self.url.href, response.headers['location']));
+            self.options.method = 'GET';
+            delete self.options.data;
+            self._retry();
+        } else {
+            self.url = url.parse(url.resolve(self.url.href, response.headers['location']));
+            self._retry();
+            // todo handle somehow infinite redirects
+        }
       } catch(err) {
         err.message = 'Failed to follow redirect: ' + err.message;
         self._fireError(err, response);
@@ -229,7 +238,6 @@ mixin(Request.prototype, {
   },
   run: function() {
     var self = this;
-    
     if (this.options.multipart) {
       multipart.write(this.request, this.options.data, function() {
         self.request.end();
@@ -326,7 +334,7 @@ function json(url, data, options, method) {
   options.parser = (typeof options.parser !== "undefined") ? options.parser : parsers.auto;
   options.headers = options.headers || {};
   options.headers['content-type'] = 'application/json';
-  options.data = JSON.stringify(data);
+  options.data = JSON.stringify(data || {});
   options.method = method || 'GET';
   return request(url, options);
 }
@@ -497,3 +505,4 @@ mixin(exports, {
   file: multipart.file,
   data: multipart.data
 });
+