feat: auto upgrade to keychain

Older repos are automatically upgraded to a keychain when the '--pass' is present.

Helps #1138

Author: richardschneider

src/core/components/no-keychain.js

@@ -1,7 +1,7 @@
 'use strict'
 
 function fail () {
-  throw new Error('Key management requires the daemon to run with \'--pass ...\'')
+  throw new Error('Key management requires \'--pass ...\' option')
 }
 
 class NoKeychain {

src/core/components/pre-start.js

@@ -13,12 +13,28 @@ module.exports = function preStart (self) {
   return (callback) => {
     self.log('pre-start')
 
+    const pass = self._options.pass
+    let importSelf = false
     waterfall([
       (cb) => self._repo.config.get(cb),
       (config, cb) => {
-        const pass = self._options.pass
+        // Upgrade to keychain?
+        if (!pass || config.Keychain) {
+          return cb(null, config)
+        }
+        config.Keychain = Keychain.generateOptions()
+        self.config.set('Keychain', config.Keychain, (err) => {
+          if (err) return cb(err)
+          const keychainOptions = Object.assign({passPhrase: pass}, config.Keychain)
+          self._keychain = new Keychain(self._repo.keys, keychainOptions)
+          importSelf = true
+          self.log('Upgrade repo for a keychain')
+          cb(null, config)
+        })
+      },
+      (config, cb) => {
         if (self._keychain) {
-          // most likely an init has happened
+          // most likely an init or upgrade has happened
         } else if (pass) {
           const keychainOptions = Object.assign({passPhrase: pass}, config.Keychain)
           self._keychain = new Keychain(self._repo.keys, keychainOptions)
@@ -32,7 +48,12 @@ module.exports = function preStart (self) {
       (config, cb) => {
         const privKey = config.Identity.PrivKey
 
-        peerId.createFromPrivKey(privKey, (err, id) => cb(err, config, id))
+        peerId.createFromPrivKey(privKey, (err, id) => {
+          if (!err && importSelf) {
+            return self._keychain.importPeer('self', id, (err) => cb(err, config, id))
+          }
+          cb(err, config, id)
+        })
       },
       (config, id, cb) => {
         self.log('peer created')