pixl-server-storage 3.1.11 → 3.1.12

package/engines/Filesystem.js

@@ -376,8 +376,9 @@ module.exports = Class.create({
 		
 		// check cache first
 		if (this.cache && this.cache.has(key)) {
+			var item = this.cache.getMeta(key);
+			
 			process.nextTick( function() {
-				var item = self.cache.getMeta(key);
 				self.logDebug(9, "Cached head complete: " + key);
 				callback( null, {
 					mod: item.date,
@@ -420,9 +421,9 @@ module.exports = Class.create({
 		
 		// check cache first
 		if (this.cache && !is_binary && this.cache.has(key)) {
+			var data = this.cache.get(key);
+			
 			process.nextTick( function() {
-				var data = self.cache.get(key);
-				
 				try { data = JSON.parse( data ); }
 				catch (e) {
 					self.logError('file', "Failed to parse JSON record: " + key + ": " + e);

package/engines/S3.js

@@ -234,8 +234,9 @@ module.exports = Class.create({
 		
 		// check cache first
 		if (this.cache && this.cache.has(orig_key)) {
+			var item = this.cache.getMeta(orig_key);
+			
 			process.nextTick( function() {
-				var item = self.cache.getMeta(orig_key);
 				self.logDebug(9, "Cached head complete: " + orig_key);
 				callback( null, {
 					mod: item.date,
@@ -285,9 +286,9 @@ module.exports = Class.create({
 		
 		// check cache first
 		if (this.cache && !is_binary && this.cache.has(orig_key)) {
-			process.nextTick( function() {
-				var data = self.cache.get(orig_key);
-				
+			var data = this.cache.get(orig_key);
+			
+			process.nextTick( function() {	
 				try { data = JSON.parse( data ); }
 				catch (e) {
 					self.logError('file', "Failed to parse JSON record: " + orig_key + ": " + e);

package/hash.js

@@ -79,12 +79,12 @@ module.exports = Class.create({
 	_hashShareLock: function(key, wait, callback) {
 		// internal hash shared lock wrapper
 		// uses unique key prefix so won't deadlock with user locks
-		this.shareLock( '|'+key, wait, callback );
+		this.shareLock( 'C|'+key, wait, callback );
 	},
 	
 	_hashShareUnlock: function(key) {
 		// internal hash shared unlock wrapper
-		this.shareUnlock( '|'+key );
+		this.shareUnlock( 'C|'+key );
 	},
 	
 	hashPut: function(path, hkey, hvalue, create_opts, callback) {

package/list.js

@@ -111,12 +111,12 @@ module.exports = Class.create({
 	_listShareLock: function(key, wait, callback) {
 		// internal list shared lock wrapper
 		// uses unique key prefix so won't deadlock with user locks
-		this.shareLock( '|'+key, wait, callback );
+		this.shareLock( 'C|'+key, wait, callback );
 	},
 	
 	_listShareUnlock: function(key) {
 		// internal list shared unlock wrapper
-		this.shareUnlock( '|'+key );
+		this.shareUnlock( 'C|'+key );
 	},
 	
 	listPush: function(key, items, create_opts, callback) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pixl-server-storage",
-	"version": "3.1.11",
+	"version": "3.1.12",
 	"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",

package/transaction.js

@@ -741,7 +741,7 @@ module.exports = Class.create({
 					// We must fsync the directory as well, as per: http://man7.org/linux/man-pages/man2/fsync.2.html
 					// Note: Yes, read-only is the only way: https://www.reddit.com/r/node/comments/4r8k11/how_to_call_fsync_on_a_directory/
 					fs.open( Path.dirname(trans.log), "r", function(err, dh) {
-						if (err) return callback(err);
+						if (err) return callback(); // this may fail on certain OSes, so treat as non-fatal
 						
 						fs.fsync(dh, function(err) {
 							// ignoring error here, as some filesystems may not allow this