backend-manager 2.2.4 → 2.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "backend-manager",
-  "version": "2.2.4",
+  "version": "2.3.0",
   "description": "Quick tools for developing Firebase functions",
   "main": "src/manager/index.js",
   "bin": {
@@ -66,4 +66,4 @@
     "src/",
     "templates/"
   ]
-}
+}

package/src/cli/cli.js

@@ -43,9 +43,9 @@ const fetch = require('node-fetch');
 
 let bem_giRegex = 'Set in .setup()'
 let bem_giRegexOuter = /# BEM>>>(.*\n?)# <<<BEM/sg;
-let bem_fsRulesRegex = /(\/\/\/---backend-manager---\/\/\/)(.*?)(\/\/\/---------end---------\/\/\/)/sgm;
-let bem_fsRulesDefaultRegex = /(\/\/\/---default-rules---\/\/\/)(.*?)(\/\/\/---------end---------\/\/\/)/sgm;
-let bem_fsRulesBackupRegex = /({{\s*?backend-manager\s*?}})/sgm;
+let bem_allRulesRegex = /(\/\/\/---backend-manager---\/\/\/)(.*?)(\/\/\/---------end---------\/\/\/)/sgm;
+let bem_allRulesDefaultRegex = /(\/\/\/---default-rules---\/\/\/)(.*?)(\/\/\/---------end---------\/\/\/)/sgm;
+let bem_allRulesBackupRegex = /({{\s*?backend-manager\s*?}})/sgm;
 let MOCHA_PKG_SCRIPT = 'mocha ../test/ --recursive --timeout=10000';
 let NPM_CLEAN_SCRIPT = 'rm -fr node_modules && rm -fr package-lock.json && npm cache clean --force && npm install && npm rb';
 let NOFIX_TEXT = chalk.red(`There is no automatic fix for this check.`);
@@ -139,7 +139,7 @@ Main.prototype.process = async function (args) {
 
   if (self.options['rules:default'] || self.options['rules:getdefault']) {
     self.getRulesFile();
-    console.log(self.default.firestoreRulesWhole.match(bem_fsRulesDefaultRegex)[0].replace('    ///', '///'));
+    console.log(self.default.firestoreRulesWhole.match(bem_allRulesDefaultRegex)[0].replace('    ///', '///'));
     return;
   }
 
@@ -205,9 +205,11 @@ module.exports = Main;
 
 Main.prototype.getRulesFile = function () {
   const self = this;
-  self.default.firestoreRulesWhole = (jetpack.read(path.resolve(`${__dirname}/../../templates/firestore.rules`))).replace('=0.0.0-', `-${self.default.version}-`);
-  self.default.firestoreRulesCore = self.default.firestoreRulesWhole.match(bem_fsRulesRegex)[0];
+  self.default.firestoreRulesWhole = (jetpack.read(path.resolve(`${__dirname}/../../templates/firestore.rules`))).replace('=0.0.0-', `=${self.default.version}-`);
+  self.default.firestoreRulesCore = self.default.firestoreRulesWhole.match(bem_allRulesRegex)[0];
 
+  self.default.databaseRulesWhole = (jetpack.read(path.resolve(`${__dirname}/../../templates/database.rules.json`))).replace('=0.0.0-', `=${self.default.version}-`);
+  self.default.databaseRulesCore = self.default.databaseRulesWhole.match(bem_allRulesRegex)[0];
 };
 
 Main.prototype.setup = async function () {
@@ -243,7 +245,7 @@ Main.prototype.setup = async function () {
 
   self.getRulesFile();
 
-  self.default.firestoreRulesVersionRegex = new RegExp(`///---version-${self.default.version}---///`)
+  self.default.rulesVersionRegex = new RegExp(`///---version-${self.default.version}---///`)
   // bem_giRegex = new RegExp(jetpack.read(path.resolve(`${__dirname}/../../templates/gitignore.md`)).replace(/\./g, '\\.'), 'm' )
   bem_giRegex = new RegExp(jetpack.read(path.resolve(`${__dirname}/../../templates/gitignore.md`)), 'm' )
 
@@ -478,22 +480,26 @@ Main.prototype.setup = async function () {
 
   // Update actual files
   await self.test('update firestore rules file', function () {
-    let exists = jetpack.exists(`${self.firebaseProjectPath}/firestore.rules`);
-    let contents = jetpack.read(`${self.firebaseProjectPath}/firestore.rules`) || '';
-    let containsCore = contents.match(bem_fsRulesRegex);
-    let matchesVersion = contents.match(self.default.firestoreRulesVersionRegex);
+    const exists = jetpack.exists(`${self.firebaseProjectPath}/firestore.rules`);
+    const contents = jetpack.read(`${self.firebaseProjectPath}/firestore.rules`) || '';
+    const containsCore = contents.match(bem_allRulesRegex);
+    const matchesVersion = contents.match(self.default.rulesVersionRegex);
 
     return (!!exists && !!containsCore && !!matchesVersion);
   }, fix_firestoreRulesFile);
 
   await self.test('update firestore indexes file', function () {
-    let exists = jetpack.exists(`${self.firebaseProjectPath}/firestore.indexes.json`);
+    const exists = jetpack.exists(`${self.firebaseProjectPath}/firestore.indexes.json`);
     return (!!exists);
   }, fix_firestoreIndexesFile);
 
   await self.test('update realtime rules file', function () {
-    let exists = jetpack.exists(`${self.firebaseProjectPath}/database.rules.json`);
-    return (!!exists);
+    const exists = jetpack.exists(`${self.firebaseProjectPath}/database.rules.json`);
+    const contents = jetpack.read(`${self.firebaseProjectPath}/database.rules.json`) || '';
+    const containsCore = contents.match(bem_allRulesRegex);
+    const matchesVersion = contents.match(self.default.rulesVersionRegex);
+
+    return (!!exists && !!containsCore && !!matchesVersion);
   }, fix_realtimeRulesFile);
 
   await self.test('update storage rules file', function () {
@@ -874,61 +880,94 @@ function fix_remoteconfigTemplate(self) {
 
 function fix_firestoreRulesFile(self) {
   return new Promise(function(resolve, reject) {
-    let path = `${self.firebaseProjectPath}/firestore.rules`;
+    const name = 'firestore.rules'
+    let path = `${self.firebaseProjectPath}/${name}`;
     let exists = jetpack.exists(path);
     let contents = jetpack.read(path) || '';
 
     if (!exists || !contents) {
-      log(chalk.yellow(`Writing new firestore.rules file...`));
+      log(chalk.yellow(`Writing new ${name} file...`));
       jetpack.write(path, self.default.firestoreRulesWhole)
       contents = jetpack.read(path) || '';
     }
 
-    let hasTemplate = contents.match(bem_fsRulesRegex) || contents.match(bem_fsRulesBackupRegex);
+    let hasTemplate = contents.match(bem_allRulesRegex) || contents.match(bem_allRulesBackupRegex);
 
     if (!hasTemplate) {
-      log(chalk.red(`Could not find rules template. Please edit firestore.rules file and add`), chalk.red(`{{backend-manager}}`), chalk.red(`to it.`));
+      log(chalk.red(`Could not find rules template. Please edit ${name} file and add`), chalk.red(`{{backend-manager}}`), chalk.red(`to it.`));
       reject()
     }
 
-    let matchesVersion = contents.match(self.default.firestoreRulesVersionRegex);
+    let matchesVersion = contents.match(self.default.rulesVersionRegex);
     if (!matchesVersion) {
       // console.log('replace wih', self.default.firestoreRulesCore);
-      contents = contents.replace(bem_fsRulesBackupRegex, self.default.firestoreRulesCore)
-      contents = contents.replace(bem_fsRulesRegex, self.default.firestoreRulesCore)
+      contents = contents.replace(bem_allRulesBackupRegex, self.default.firestoreRulesCore)
+      contents = contents.replace(bem_allRulesRegex, self.default.firestoreRulesCore)
       jetpack.write(path, contents)
-      log(chalk.yellow(`Writing core rules to firestore.rules file...`));
+      log(chalk.yellow(`Writing core rules to ${name} file...`));
     }
     resolve();
   });
 };
 
-function fix_firestoreIndexesFile(self) {
-  return new Promise(async function(resolve, reject) {
-    const name = 'firestore.indexes.json';
-    let filePath = `${self.firebaseProjectPath}/${name}`;
-    let exists = jetpack.exists(filePath);
+function fix_realtimeRulesFile(self) {
+  return new Promise(function(resolve, reject) {
+    const name = 'database.rules.json'
+    let path = `${self.firebaseProjectPath}/${name}`;
+    let exists = jetpack.exists(path);
+    let contents = jetpack.read(path) || '';
 
-    if (!exists) {
+    if (!exists || !contents) {
       log(chalk.yellow(`Writing new ${name} file...`));
-      await cmd_indexesGet(self, name, false);
+      jetpack.write(path, self.default.databaseRulesWhole)
+      contents = jetpack.read(path) || '';
+    }
+
+    let hasTemplate = contents.match(bem_allRulesRegex) || contents.match(bem_allRulesBackupRegex);
+
+    if (!hasTemplate) {
+      log(chalk.red(`Could not find rules template. Please edit ${name} file and add`), chalk.red(`{{backend-manager}}`), chalk.red(`to it.`));
+      reject()
     }
 
+    let matchesVersion = contents.match(self.default.rulesVersionRegex);
+    if (!matchesVersion) {
+      // console.log('replace wih', self.default.databaseRulesCore);
+      contents = contents.replace(bem_allRulesBackupRegex, self.default.databaseRulesCore)
+      contents = contents.replace(bem_allRulesRegex, self.default.databaseRulesCore)
+      jetpack.write(path, contents)
+      log(chalk.yellow(`Writing core rules to ${name} file...`));
+    }
     resolve();
   });
 };
 
-function fix_realtimeRulesFile(self) {
-  return new Promise(function(resolve, reject) {
-    const name = 'database.rules.json';
+// function fix_realtimeRulesFile(self) {
+//   return new Promise(function(resolve, reject) {
+//     const name = 'database.rules.json';
+//     let filePath = `${self.firebaseProjectPath}/${name}`;
+//     let exists = jetpack.exists(filePath);
+//     let contents = jetpack.read(filePath) || '';
+//
+//     if (!exists) {
+//       log(chalk.yellow(`Writing new ${name} file...`));
+//       jetpack.write(filePath, jetpack.read(path.resolve(`${__dirname}/../../templates/${name}`)))
+//       contents = jetpack.read(filePath) || '';
+//     }
+//
+//     resolve();
+//   });
+// };
+
+function fix_firestoreIndexesFile(self) {
+  return new Promise(async function(resolve, reject) {
+    const name = 'firestore.indexes.json';
     let filePath = `${self.firebaseProjectPath}/${name}`;
     let exists = jetpack.exists(filePath);
-    let contents = jetpack.read(filePath) || '';
 
     if (!exists) {
       log(chalk.yellow(`Writing new ${name} file...`));
-      jetpack.write(filePath, jetpack.read(path.resolve(`${__dirname}/../../templates/${name}`)))
-      contents = jetpack.read(filePath) || '';
+      await cmd_indexesGet(self, name, false);
     }
 
     resolve();

package/src/manager/functions/core/actions/api/handler/create-post.js

@@ -29,7 +29,7 @@ Module.prototype.main = function () {
       }
     }
 
-    const postUrl = `blog/${payload.data.payload.url}`;
+    const postUrl = `${Manager.config.brand.url}/blog/${payload.data.payload.url}`;
 
     if (payload.data.payload.invoiceEmail && payload.data.payload.invoicePrice) {
       // Create invoice

package/templates/database.rules.json

@@ -1,31 +1,46 @@
 {
   "rules": {
+    ///---backend-manager---///
+    ///---version=0.0.0---///
+
+    // Gathering rules
     "gatherings": {
       ".read": false,
       ".write": false,
-			"online": {
-        ".read": "auth.uid != null && query.equalTo == auth.uid",
+			"$room": {
+        ".read": "
+        	(auth.uid != null && query.equalTo == auth.uid)
+        ",
         ".write": false,
         ".indexOn": ["uid"],
-        "$uid": {
+        "$id": {
           ".read": "
-          	// Allowed if user is signed in AND is the owner of the doc
+          	// Allowed if user is authenticated AND is the owner of the doc
           	(auth != null && auth.uid == data.child('uid').val())
+            // Allowed if uid is equal to the doc id [LEGACY FOR SOMIIBO]
+            || (auth != null && auth.uid == $id)
+          	// Allowed if user is not authenticated AND is the doc has no owner
+          	|| (auth == null && (data.child('uid').val() == ''))
           ",
           ".write": "
           	// Allowed if the user is signed in AND is the owner of the existing doc
           	(auth != null && auth.uid == data.child('uid').val())
           	// Allowed if the user is signed in AND is the owner of the new doc
           	|| (auth != null && auth.uid == newData.child('uid').val())
-          	// Allowed if it's a delete
-          	|| (!newData.exists())
-						// Allowed if the existing doc has no owner
+            // Allowed if the user is signed in AND is the owner of the existing doc
+          	|| (auth != null && auth.uid == data.child('uid').val())
+            // Allowed if uid is equal to the doc id [LEGACY FOR SOMIIBO]
+            || (auth != null && auth.uid == $id)
+            // Allowed if the existing doc has no owner
           	|| (data.child('uid').val() == '')
           	// Allowed if the new doc has no owner
           	|| (newData.child('uid').val() == '')
+            // Allowed if it's a delete
+          	|| (!newData.exists())
           ",
       	}
       }
-    }
+    },
+    ///---------end---------///
   }
 }