Extend U.request() + RESTBuilder by adding persisten cookies.

builders.js

@@ -3659,6 +3659,7 @@ function RESTBuilder(url) {
 	this.$length = 0;
 	this.$transform = transforms['restbuilder_default'];
 	this.$files = null;
+	this.$persistentcookies = false;
 
 	// this.$flags;
 	// this.$data = {};
@@ -3894,9 +3895,18 @@ RESTBuilder.prototype.raw = function(value) {
 	return this;
 };
 
+RESTBuilder.prototype.cook = function(value) {
+	this.$persistentcookies = value !== false;
+	return this;
+};
+
+RESTBuilder.prototype.cookies = function(obj) {
+	this.$cookies = obj;
+	return this;
+};
+
 RESTBuilder.prototype.cookie = function(name, value) {
-	if (!this.$cookies)
-		this.$cookies = {};
+	!this.$cookies && (this.$cookies = {});
 	this.$cookies[name] = value;
 	return this;
 };

changes.txt

@@ -6,10 +6,13 @@
 - added: `LOGMAIL()` global alias for `F.logmail()`
 - added: `MAIL()` global alias for `F.mail()`
 - added: own and improved implementation of `onFinished`
+- added: `RESTBuilder.cookies(obj)` can set cookies as raw object
+- added: `RESTBuilder.cook([true/false])` enables persistent cookies
 
 - updated: `F.load()`, now supports `string` for `debug` or `release` mode
 - updated: `F.cluster.request()` can be executed from master process
 - updated: `Image.miniature()` change a default filter from `Box` to `Hamming`
+- updated: `U.request()` supports a new flag `cookies` which enables a parsing cookies from response
 
 - fixed: schema validation (problem with Arrays)
 - fixed: determines `x-forwarded-proto`

utils.js

@@ -172,6 +172,7 @@ var CONTENTTYPES = {
 	'zip': 'application/zip'
 };
 
+var persistentcookies = {};
 var dnscache = {};
 var datetimeformat = {};
 const hasOwnProperty = Object.prototype.hasOwnProperty;
@@ -435,6 +436,7 @@ exports.request = function(url, flags, data, callback, cookies, headers, encodin
 	var options = { length: 0, timeout: timeout || 10000, evt: new EventEmitter2(), encoding: typeof(encoding) !== 'string' ? ENCODING : encoding, callback: callback, post: false, redirect: 0 };
 	var method;
 	var type = 0;
+	var isCookies = false;
 
 	if (headers)
 		headers = exports.extend({}, headers);
@@ -518,7 +520,7 @@ exports.request = function(url, flags, data, callback, cookies, headers, encodin
 					break;
 
 				case 'cookies':
-					options.cookies = cookies;
+					isCookies = true;
 					break;
 			}
 		}
@@ -548,6 +550,8 @@ exports.request = function(url, flags, data, callback, cookies, headers, encodin
 		options.data = data;
 
 	if (cookies) {
+		if (isCookies)
+			options.cookies = cookies;
 		var builder = '';
 		for (var m in cookies)
 			builder += (builder ? '; ' : '') + m + '=' + cookies[m];
@@ -560,8 +564,6 @@ exports.request = function(url, flags, data, callback, cookies, headers, encodin
 	uri.agent = false;
 	uri.headers = headers;
 
-	if (options.cookies)
-
 	if (options.resolve) {
 		exports.resolve(url, function(err, u) {
 			!err && (uri.host = u.host);
@@ -726,12 +728,23 @@ function request_response(res, uri, options) {
 
 	// Shared cookies
 	if (options.cookies) {
-		var arr = (res.headers['cookie'] || '').split(';');
-		for (var i = 0, length = arr.length; i < length; i++) {
-			var line = arr[i].trim();
-			var index = line.indexOf('=');
-			if (index !== -1)
-				options.cookies[line.substring(0, index)] = decodeURIComponent(line.substring(index + 1));
+		var arr = (res.headers['set-cookie'] || '');
+
+		// Only the one value
+		if (arr && !(arr instanceof Array))
+			arr = [arr];
+
+		if (arr instanceof Array) {
+			for (var i = 0, length = arr.length; i < length; i++) {
+				var line = arr[i];
+				var end = line.indexOf(';');
+				if (end === -1)
+					end = line.length;
+				line = line.substring(0, end);
+				var index = line.indexOf('=');
+				if (index !== -1)
+					options.cookies[line.substring(0, index)] = decodeURIComponent(line.substring(index + 1));
+			}
 		}
 	}
 
@@ -907,7 +920,6 @@ exports.download = function(url, flags, data, callback, cookies, headers, encodi
 				case 'dnscache':
 					options.resolve = true;
 					break;
-
 			}
 		}
 	}