actually use push target filters when new books are published

api/controllers/BooksController.js

@@ -7,6 +7,7 @@
 
 const HttpError = require('../errors/HttpError')
 const request = require('request')
+const uriRegex = /^(.+:\/\/)?(.+\.)*(.+\.).{1,}(:\d+)?(.+)?/i
 
 module.exports = {
   publish: async function (req, res) {
@@ -73,16 +74,27 @@ module.exports = {
 async function sendUpdatesAsync (id) {
   const book = await Book.findOne({ id })
   const targets = await TargetUrl.find()
+  if (!book) return
   for (const i in targets) {
-    sails.log('sending ' + book.id + ' info to ' + targets[i].url)
+    const item = targets[i]
-    request.post({
+    const { author: fAuthor, publisher: fPublisher, title: fTitle, isbn: fIsbn, url } = item
-      url: targets[i].url,
+    const { author: bAuthor, publisher: bPublisher, title: bTitle, isbn: bIsbn } = book
-      headers: { 'User-Agent': 'RoE-aggregator' },
+    sails.log('sending ' + book.id + ' info to ' + url)
-      form: book
+
-    }, function (err, httpResp, body) {
+    if (uriRegex.test(url)) {
-      if (err) {
+      if (fAuthor && !((bAuthor || '').includes(fAuthor))) continue
-        sails.log(`error: failed to send book ${id} to ${targets[i].url}`)
+      if (fPublisher && !((bPublisher || '').includes(fPublisher))) continue
-      }
+      if (fTitle && !((bTitle || '').includes(fTitle))) continue
-    })
+      if (fIsbn && !((bIsbn || '').includes(fIsbn))) continue
+      request.post({
+        url: url,
+        headers: { 'User-Agent': 'RoE-aggregator' },
+        form: book
+      }, function (err, httpResp, body) {
+        if (err) {
+          sails.log(`error: failed to send book ${id} to ${url}`)
+        }
+      })
+    }
   }
 }

api/models/Book.js

@@ -19,6 +19,7 @@ module.exports = {
     },
     title: { type: 'string', required: true },
     author: { type: 'string' },
+    publisher: { type: 'string' },
     isbn: { type: 'string' },
     version: { type: 'string' }
 

migrations/20190220123908_AddPublisherToBooks.js

@@ -0,0 +1,15 @@
+exports.up = function (knex, Promise) {
+  return Promise.all([
+    knex.schema.table('book', t => {
+      t.string('publisher')
+    })
+  ])
+}
+
+exports.down = function (knex, Promise) {
+  return Promise.all([
+    knex.schema.table('book', t => {
+      t.dropColumns('publisher')
+    })
+  ])
+}

package.json

@@ -21,7 +21,8 @@
     "debug": "node --inspect app.js",
     "custom-tests": "echo 'Nothing yet'",
     "db:migrate": "knex migrate:latest",
-    "db:rollback": "knex migrate:rollback"
+    "db:rollback": "knex migrate:rollback",
+    "db:gmig": "knex migrate:make"
   },
   "dependencies": {
     "@sailshq/connect-redis": "^3.2.1",