Error handling mini refractoring

[github/Chocobozzz/PeerTube.git] / controllers / api / v1 / pods.js

diff --git a/controllers/api/v1/pods.js b/controllers/api/v1/pods.js
index 30385bd5a7bd438c3577b2075383cd5f927e097c..456f53dea0fb88be4fa491412f91f477985c18ca 100644 (file)
--- a/controllers/api/v1/pods.js
+++ b/controllers/api/v1/pods.js
@@ -2,68 +2,94 @@
   'use strict'
 
   var express = require('express')
-  var router = express.Router()
+  var fs = require('fs')
+
+  var logger = require('../../../helpers/logger')
+  var friends = require('../../../lib/friends')
   var middleware = require('../../../middlewares')
-  var miscMiddleware = middleware.misc
+  var cacheMiddleware = middleware.cache
+  var peertubeCrypto = require('../../../helpers/peertubeCrypto')
+  var Pods = require('../../../models/pods')
   var reqValidator = middleware.reqValidators.pods
+  var secureMiddleware = middleware.secure
   var secureRequest = middleware.reqValidators.remote.secureRequest
-  var pods = require('../../../models/pods')
+  var Videos = require('../../../models/videos')
 
-  function listPods (req, res, next) {
-    pods.list(function (err, pods_list) {
+  var router = express.Router()
+
+  router.get('/', cacheMiddleware.cache(false), listPods)
+  router.post('/', reqValidator.podsAdd, cacheMiddleware.cache(false), addPods)
+  router.get('/makefriends', reqValidator.makeFriends, cacheMiddleware.cache(false), makeFriends)
+  router.get('/quitfriends', cacheMiddleware.cache(false), quitFriends)
+  // Post because this is a secured request
+  router.post('/remove', secureRequest, secureMiddleware.decryptBody, removePods)
+
+  // ---------------------------------------------------------------------------
+
+  module.exports = router
+
+  // ---------------------------------------------------------------------------
+
+  function addPods (req, res, next) {
+    var informations = req.body.data
+    Pods.add(informations, function (err) {
       if (err) return next(err)
 
-      res.json(pods_list)
+      Videos.addRemotes(informations.videos)
+
+      fs.readFile(peertubeCrypto.getCertDir() + 'peertube.pub', 'utf8', function (err, cert) {
+        if (err) {
+          logger.error('Cannot read cert file.')
+          return next(err)
+        }
+
+        Videos.listOwned(function (err, videos_list) {
+          if (err) {
+            logger.error('Cannot get the list of owned videos.')
+            return next(err)
+          }
+
+          res.json({ cert: cert, videos: videos_list })
+        })
+      })
     })
   }
 
-  function addPods (req, res, next) {
-    pods.add(req.body.data, function (err, json) {
+  function listPods (req, res, next) {
+    Pods.list(function (err, pods_list) {
       if (err) return next(err)
 
-      res.json(json)
+      res.json(pods_list)
     })
   }
 
-  function removePods (req, res, next) {
-    pods.remove(req.body.signature.url, function (err) {
+  function makeFriends (req, res, next) {
+    friends.makeFriends(function (err) {
       if (err) return next(err)
 
       res.sendStatus(204)
     })
   }
 
-  function makeFriends (req, res, next) {
-    pods.hasFriends(function (err, has_friends) {
+  function removePods (req, res, next) {
+    var url = req.body.signature.url
+    Pods.remove(url, function (err) {
       if (err) return next(err)
 
-      if (has_friends === true) {
-        // We need to quit our friends before make new ones
-        res.sendStatus(409)
-      } else {
-        pods.makeFriends(function (err) {
-          if (err) return next(err)
+      Videos.removeAllRemotesOf(url, function (err) {
+        if (err) logger.error('Cannot remove all remote videos of %s.', url)
+        else logger.info('%s pod removed.', url)
 
-          res.sendStatus(204)
-        })
-      }
+        res.sendStatus(204)
+      })
     })
   }
 
   function quitFriends (req, res, next) {
-    pods.quitFriends(function (err) {
+    friends.quitFriends(function (err) {
       if (err) return next(err)
 
       res.sendStatus(204)
     })
   }
-
-  router.get('/', miscMiddleware.cache(false), listPods)
-  router.get('/makefriends', miscMiddleware.cache(false), makeFriends)
-  router.get('/quitfriends', miscMiddleware.cache(false), quitFriends)
-  router.post('/', reqValidator.podsAdd, miscMiddleware.cache(false), addPods)
-  // Post because this is a secured request
-  router.post('/remove', secureRequest, miscMiddleware.decryptBody, removePods)
-
-  module.exports = router
 })()