pixl-server-storage 3.1.13 → 3.1.15

package/docs/API.md

@@ -17,6 +17,7 @@ var storage = server.Storage;
 	* [putStream](#putstream)
 	* [get](#get)
 	* [getMulti](#getmulti)
+	* [getBuffer](#getbuffer)
 	* [getStream](#getstream)
 	* [getStreamRange](#getstreamrange)
 	* [head](#head)
@@ -60,6 +61,8 @@ var storage = server.Storage;
 	* [hashPutMulti](#hashputmulti)
 	* [hashGet](#hashget)
 	* [hashGetMulti](#hashgetmulti)
+	* [hashUpdate](#hashupdate)
+	* [hashUpdateMulti](#hashupdatemulti)
 	* [hashEach](#hasheach)
 	* [hashEachSync](#hasheachsync)
 	* [hashEachPage](#hasheachpage)
@@ -1012,6 +1015,28 @@ storage.hashUpdate( 'users', 'bsanders', { age: 81 }, function(err) {
 
 This would update Bernie's age to 81 without affecting any of the other properties in his user record.
 
+## hashUpdateMulti
+
+```javascript
+storage.hashUpdateMulti( PATH, RECORDS, CALLBACK );
+```
+
+The `hashUpdateMulti()` method updates multiple key/value pairs in a hash.  The `PATH` specifies the main storage path of the hash, and `RECORDS` should be an object containing all the keys and values you want to update.  See [hashUpdate()](#hashupdate) for details on the update format.
+
+```js
+var updates = {
+	"bsanders": { age: 81 },
+	"hclinton": { age: 75 },
+	"dtrump": { age: 77 }
+};
+
+storage.hashUpdateMulti( 'users', records, function(err) {
+	if (err) throw err;
+} );
+```
+
+This would update the `age` properties in each record, without affecting the other data.
+
 ## hashEach
 
 ```javascript

package/docs/Transactions.md

@@ -199,8 +199,9 @@ When transactions are enabled, many compound storage methods (those that execute
 - **Hash Methods**
 	- [hashCreate()](API.md#hashcreate)
 	- [hashPut()](API.md#hashput)
-	- [hashUpdate()](API.md#hashupdate)
 	- [hashPutMulti()](API.md#hashputmulti)
+	- [hashUpdate()](API.md#hashupdate)
+	- [hashUpdateMulti()](API.md#hashupdatemulti)
 	- [hashCopy()](API.md#hashcopy)
 	- [hashRename()](API.md#hashrename)
 	- [hashDeleteMulti()](API.md#hashdeletemulti)

package/engines/S3.js

@@ -10,6 +10,7 @@ var S3 = require("@aws-sdk/client-s3");
 var { Upload } = require("@aws-sdk/lib-storage");
 var { NodeHttpHandler } = require("@smithy/node-http-handler");
 var streamToBuffer = require("fast-stream-to-buffer");
+var StreamMeter = require("stream-meter");
 
 module.exports = Class.create({
 	
@@ -100,8 +101,6 @@ module.exports = Class.create({
 	
 	prepKey: function(key) {
 		// prepare key for S3 based on config
-		var md5 = Tools.digestHex(key, 'md5');
-		
 		var ns = '';
 		if (key.match(/^([\w\-\.]+)\//)) ns = RegExp.$1;
 		
@@ -110,6 +109,7 @@ module.exports = Class.create({
 		}
 		
 		if (this.keyTemplate) {
+			var md5 = Tools.digestHex(key, 'md5');
 			var idx = 0;
 			var temp = this.keyTemplate.replace( /\#/g, function() {
 				return md5.substr(idx++, 1);
@@ -153,6 +153,7 @@ module.exports = Class.create({
 		this.s3.send( new S3.PutObjectCommand(params) )
 			.then( function(data) {
 				self.logDebug(9, "Store complete: " + key);
+				self.storage.emit('billing', 's3_put', 1);
 				
 				// possibly cache in LRU
 				if (self.cache && !is_binary) {
@@ -162,6 +163,13 @@ module.exports = Class.create({
 				if (callback) process.nextTick( function() { callback(null, data); });
 			} )
 			.catch( function(err) {
+				if (err.name == 'SlowDown') {
+					// special behavior for SlowDown errors
+					self.logDebug(6, "Received SlowDown from S3 put: " + orig_key + ": " + err + " (will retry)");
+					self.storage.emit('slowDown');
+					setTimeout( function() { self.put(orig_key, value, callback); }, 1000 );
+					return;
+				}
 				self.logError('s3', "Failed to store object: " + key + ": " + (err.message || err), err);
 				if (callback) process.nextTick( function() { callback(err); });
 			} );
@@ -173,9 +181,12 @@ module.exports = Class.create({
 		var orig_key = key;
 		key = this.prepKey(key);
 		
+		var meter = new StreamMeter();
+		inp.pipe(meter);
+		
 		var params = Tools.copyHash( this.s3Params );
 		params.Key = this.extKey(key, orig_key);
-		params.Body = inp;
+		params.Body = meter;
 		
 		this.logDebug(9, "Storing S3 Binary Stream: " + key);
 		
@@ -187,6 +198,8 @@ module.exports = Class.create({
 		upload.done()
 			.then( function(data) {
 				self.logDebug(9, "Stream store complete: " + key);
+				self.storage.emit('billing', 's3_put', 1);
+				self.storage.emit('billing', 's3_bytes_out', meter.bytes);
 				if (callback) process.nextTick( function() { callback(null, data); });
 			} )
 			.catch( function(err) {
@@ -201,9 +214,12 @@ module.exports = Class.create({
 		var orig_key = key;
 		key = this.prepKey(key);
 		
+		var meter = new StreamMeter();
+		inp.pipe(meter);
+		
 		var params = Tools.copyHash( this.s3Params );
 		params.Key = this.extKey(key, orig_key);
-		params.Body = inp;
+		params.Body = meter;
 		if (opts) Tools.mergeHashInto(params, opts);
 		
 		this.logDebug(9, "Storing S3 Binary Stream: " + key);
@@ -216,6 +232,8 @@ module.exports = Class.create({
 		upload.done()
 			.then( function(data) {
 				self.logDebug(9, "Stream store complete: " + key);
+				self.storage.emit('billing', 's3_put', 1);
+				self.storage.emit('billing', 's3_bytes_out', meter.bytes);
 				if (callback) process.nextTick( function() { callback(null, data); });
 			} )
 			.catch( function(err) {
@@ -252,6 +270,7 @@ module.exports = Class.create({
 		this.s3.send( new S3.HeadObjectCommand(params) )
 			.then( function(data) {
 				self.logDebug(9, "Head complete: " + key);
+				self.storage.emit('billing', 's3_head', 1);
 				
 				process.nextTick( function() {
 					callback( null, {
@@ -267,6 +286,13 @@ module.exports = Class.create({
 					err = new Error("Failed to head key: " + key + ": Not found");
 					err.code = "NoSuchKey";
 				}
+				else if (err.name == 'SlowDown') {
+					// special behavior for SlowDown errors
+					self.logDebug(6, "Received SlowDown from S3 head: " + orig_key + ": " + err + " (will retry)");
+					self.storage.emit('slowDown');
+					setTimeout( function() { self.head(orig_key, callback); }, 1000 );
+					return;
+				}
 				else {
 					// some other error
 					self.logError('s3', "Failed to head key: " + key + ": " + (err.message || err), err);
@@ -314,6 +340,9 @@ module.exports = Class.create({
 						return callback(err);
 					}
 					
+					self.storage.emit('billing', 's3_get', 1);
+					self.storage.emit('billing', 's3_bytes_in', body.length);
+					
 					if (is_binary) {
 						self.logDebug(9, "Binary fetch complete: " + key, '' + body.length + ' bytes');
 					}
@@ -347,6 +376,13 @@ module.exports = Class.create({
 					err = new Error("Failed to fetch key: " + key + ": Not found");
 					err.code = "NoSuchKey";
 				}
+				else if (err.name == 'SlowDown') {
+					// special behavior for SlowDown errors
+					self.logDebug(6, "Received SlowDown from S3 get: " + orig_key + ": " + err + " (will retry)");
+					self.storage.emit('slowDown');
+					setTimeout( function() { self.get(orig_key, callback); }, 1000 );
+					return;
+				}
 				else {
 					// some other error
 					self.logError('s3', "Failed to fetch key: " + key + ": " + (err.message || err), err);
@@ -376,6 +412,8 @@ module.exports = Class.create({
 					}
 					
 					self.logDebug(9, "Binary fetch complete: " + key, '' + body.length + ' bytes');
+					self.storage.emit('billing', 's3_get', 1);
+					self.storage.emit('billing', 's3_bytes_in', body.length);
 					
 					callback( null, body, {
 						mod: Math.floor((new Date(data.LastModified)).getTime() / 1000),
@@ -390,6 +428,13 @@ module.exports = Class.create({
 					err = new Error("Failed to fetch key: " + key + ": Not found");
 					err.code = "NoSuchKey";
 				}
+				else if (err.name == 'SlowDown') {
+					// special behavior for SlowDown errors
+					self.logDebug(6, "Received SlowDown from S3 getBuffer: " + orig_key + ": " + err + " (will retry)");
+					self.storage.emit('slowDown');
+					setTimeout( function() { self.getBuffer(orig_key, callback); }, 1000 );
+					return;
+				}
 				else {
 					// some other error
 					self.logError('s3', "Failed to fetch key: " + key + ": " + (err.message || err), err);
@@ -423,6 +468,9 @@ module.exports = Class.create({
 					self.logDebug(9, "S3 stream download closed: " + key);
 				} );
 				
+				self.storage.emit('billing', 's3_get', 1);
+				self.storage.emit('billing', 's3_bytes_in', data.ContentLength);
+				
 				process.nextTick( function() {
 					callback( null, download, {
 						mod: Math.floor( (new Date(data.LastModified)).getTime() / 1000 ),
@@ -478,6 +526,9 @@ module.exports = Class.create({
 					self.logDebug(9, "S3 stream download closed: " + key);
 				} );
 				
+				self.storage.emit('billing', 's3_get', 1);
+				self.storage.emit('billing', 's3_bytes_in', data.ContentRange);
+				
 				// get full length from the ContentRange header
 				var len = 0;
 				if (data.ContentRange && data.ContentRange.toString().match(/\/\s*(\d+)\s*$/)) {
@@ -521,6 +572,7 @@ module.exports = Class.create({
 		this.s3.send( new S3.DeleteObjectCommand(params) )
 			.then( function(data) {
 				self.logDebug(9, "Delete complete: " + key);
+				self.storage.emit('billing', 's3_delete', 1);
 				
 				// possibly delete from LRU cache as well
 				if (self.cache && self.cache.has(orig_key)) {
@@ -530,6 +582,13 @@ module.exports = Class.create({
 				if (callback) process.nextTick( function() { callback(null, data); } );
 			} )
 			.catch( function(err) {
+				if (err.name == 'SlowDown') {
+					// special behavior for SlowDown errors
+					self.logDebug(6, "Received SlowDown from S3 delete: " + orig_key + ": " + err + " (will retry)");
+					self.storage.emit('slowDown');
+					setTimeout( function() { self.delete(orig_key, callback); }, 1000 );
+					return;
+				}
 				self.logError('s3', "Failed to delete object: " + key + ": " + (err.message || err), err);
 				if (callback) process.nextTick( function() { callback(err); } );
 			} );

package/hash.js

@@ -426,6 +426,25 @@ module.exports = Class.create({
 		}); // get
 	},
 	
+	hashUpdateMulti: function(path, records, callback) {
+		// update multiple hash records at once, given object of keys and values
+		// need concurrency limit of 1 because hashUpdate locks
+		var self = this;
+		
+		async.eachLimit(Object.keys(records), 1, 
+			function(hkey, callback) {
+				// iterator for each key
+				self.hashUpdate(path, hkey, records[hkey], function(err) {
+					callback(err);
+				} );
+			}, 
+			function(err) {
+				// all keys updated
+				callback(err);
+			}
+		);
+	},
+	
 	hashEachPage: function(path, iterator, callback) {
 		// call user iterator for each populated hash page, data only
 		// iterator will be passed page items hash object

package/indexer.js

@@ -75,7 +75,7 @@ module.exports = Class.create({
 	validateIndexConfig: function(config) {
 		// make sure index config is kosher
 		// return false for success, or error on failure
-		if (!config || !config.fields || !config.fields.length) {
+		if (!config || !config.fields || !Tools.isaArray(config.fields)) {
 			return( new Error("Invalid index configuration object.") );
 		}
 		if (Tools.findObject(config.fields, { _primary: 1 })) {
@@ -104,6 +104,10 @@ module.exports = Class.create({
 		
 		// validate each sorter def
 		if (config.sorters) {
+			if (!Tools.isaArray(config.sorters)) {
+				return( new Error("Invalid index sorters array.") );
+			}
+			
 			for (var idx = 0, len = config.sorters.length; idx < len; idx++) {
 				var sorter = config.sorters[idx];
 				
@@ -163,18 +167,12 @@ module.exports = Class.create({
 		var fields = [];
 		
 		config.fields.forEach( function(def) {
-			var value = def.source.match(/^\//) ? Tools.lookupPath(def.source, record) : Tools.sub(def.source, record, true);
+			var value = def.source.match(/\[.+\]/) ? Tools.sub(def.source, record, true) : Tools.getPath(record, def.source);
+			if (value === undefined) value = null;
 			if ((value === null) && ("default_value" in def)) value = def.default_value;
 			if (value !== null) fields.push(def);
 		} );
 		
-		if (!fields.length) {
-			// nothing to index!
-			this.logDebug(6, "Nothing to index, skipping entire record");
-			if (callback) callback();
-			return;
-		}
-		
 		// start index and track perf
 		var pf = this.perf.begin('index');
 		
@@ -211,7 +209,8 @@ module.exports = Class.create({
 							return;
 						}
 						
-						var value = def.source.match(/^\//) ? Tools.lookupPath(def.source, record) : Tools.sub(def.source, record, true);
+						var value = def.source.match(/\[.+\]/) ? Tools.sub(def.source, record, true) : Tools.getPath(record, def.source);
+						if (value === undefined) value = null;
 						if ((value === null) && ("default_value" in def)) value = def.default_value;
 						if (typeof(value) == 'object') value = JSON.stringify(value);
 						
@@ -920,7 +919,8 @@ module.exports = Class.create({
 		// add record to sorter index
 		var config = state.config;
 		
-		var value = Tools.lookupPath(sorter.source, record);
+		var value = Tools.getPath(record, sorter.source);
+		if (value === undefined) value = null;
 		if ((value === null) && ("default_value" in sorter)) value = sorter.default_value;
 		if (value === null) {
 			if (state.new_record) value = ((sorter.type == 'number') ? 0 : '');

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pixl-server-storage",
-	"version": "3.1.13",
+	"version": "3.1.15",
 	"description": "A key/value/list storage component for the pixl-server framework.",
 	"author": "Joseph Huckaby <jhuckaby@gmail.com>",
 	"homepage": "https://github.com/jhuckaby/pixl-server-storage",
@@ -36,6 +36,7 @@
 		"pixl-server": "^1.0.39",
 		"pixl-tools": "^1.0.33",
 		"porter-stemmer": "0.9.1",
+		"stream-meter": "1.0.4",
 		"unidecode": "0.1.8"
 	},
 	"devDependencies": {

package/transaction.js

@@ -454,8 +454,9 @@ module.exports = Class.create({
 			'listEachPageUpdate',
 			'hashCreate', 
 			'hashPut', 
+			'hashPutMulti',
 			'hashUpdate',
-			'hashPutMulti', 
+			'hashUpdateMulti', 
 			'hashCopy', 
 			'hashRename', 
 			'hashDeleteMulti',