Add plugin table migration table

diff --git a/server/initializers/constants.ts b/server/initializers/constants.ts
index 1111fd97f3884ba5b030a2d942b8dfbde8fb48df..5329d5fabb18d9b14f66bee52cc0a36a2c691364 100644 (file)
--- a/server/initializers/constants.ts
+++ b/server/initializers/constants.ts
@@ -14,7 +14,7 @@ import { CONFIG, registerConfigChangedHandler } from './config'
 
 // ---------------------------------------------------------------------------
 
-const LAST_MIGRATION_VERSION = 400
+const LAST_MIGRATION_VERSION = 405
 
 // ---------------------------------------------------------------------------
 

diff --git a/server/initializers/migrations/0405-plugin.ts b/server/initializers/migrations/0405-plugin.ts
new file mode 100644 (file)
index 0000000..c55b819
--- /dev/null
+++ b/server/initializers/migrations/0405-plugin.ts
@@ -0,0 +1,40 @@
+import * as Sequelize from 'sequelize'
+
+async function up (utils: {
+  transaction: Sequelize.Transaction,
+  queryInterface: Sequelize.QueryInterface,
+  sequelize: Sequelize.Sequelize,
+  db: any
+}): Promise<void> {
+  {
+    const query = `
+CREATE TABLE IF NOT EXISTS "plugin"
+(
+  "id"             SERIAL,
+  "name"           VARCHAR(255)             NOT NULL,
+  "type"           INTEGER                  NOT NULL,
+  "version"        VARCHAR(255)             NOT NULL,
+  "latestVersion"  VARCHAR(255),
+  "enabled"        BOOLEAN                  NOT NULL,
+  "uninstalled"    BOOLEAN                  NOT NULL,
+  "peertubeEngine" VARCHAR(255)             NOT NULL,
+  "description"    VARCHAR(255),
+  "homepage"       VARCHAR(255)             NOT NULL,
+  "settings"       JSONB,
+  "storage"        JSONB,
+  "createdAt"      TIMESTAMP WITH TIME ZONE NOT NULL,
+  "updatedAt"      TIMESTAMP WITH TIME ZONE NOT NULL,
+  PRIMARY KEY ("id")
+);`
+    await utils.sequelize.query(query)
+  }
+}
+
+function down (options) {
+  throw new Error('Not implemented.')
+}
+
+export {
+  up,
+  down
+}

diff --git a/server/lib/plugins/hooks.ts b/server/lib/plugins/hooks.ts
index b694d4118f9a991695dfcb6ec602bcbfd9e1941f..bcc8c674ec443b844894dcdcb47da1fa25c22909 100644 (file)
--- a/server/lib/plugins/hooks.ts
+++ b/server/lib/plugins/hooks.ts
@@ -9,7 +9,7 @@ type RawFunction <U, T> = (params: U) => T
 // Helpers to run hooks
 const Hooks = {
   wrapObject: <T, U extends ServerFilterHookName>(result: T, hookName: U) => {
-    return PluginManager.Instance.runHook(hookName, result) as Promise<T>
+    return PluginManager.Instance.runHook(hookName, result)
   },
 
   wrapPromiseFun: async <U, T, V extends ServerFilterHookName>(fun: PromiseFunction<U, T>, params: U, hookName: V) => {

diff --git a/server/lib/plugins/plugin-manager.ts b/server/lib/plugins/plugin-manager.ts
index 6485a47c515a8304d00fec02c2366bc59f8ed6be..afc07a1511ccab86d87144575ba78e7fcb6d2a8d 100644 (file)
--- a/server/lib/plugins/plugin-manager.ts
+++ b/server/lib/plugins/plugin-manager.ts
@@ -125,6 +125,13 @@ export class PluginManager implements ServerHook {
       try {
         await this.registerPluginOrTheme(plugin)
       } catch (err) {
+        // Try to unregister the plugin
+        try {
+          await this.unregister(PluginModel.buildNpmName(plugin.name, plugin.type))
+        } catch {
+          // we don't care if we cannot unregister it
+        }
+
         logger.error('Cannot register plugin %s, skipping.', plugin.name, { err })
       }
     }

diff --git a/server/tests/plugins/action-hooks.ts b/server/tests/plugins/action-hooks.ts
index 2a941148a42ce102409fa99dea72b20df914dcbc..e28732caca079ec3367edeb6448a0695f5d6c229 100644 (file)
--- a/server/tests/plugins/action-hooks.ts
+++ b/server/tests/plugins/action-hooks.ts
@@ -43,7 +43,7 @@ describe('Test plugin action hooks', function () {
       path: getPluginTestPath()
     })
 
-    await killallServers([ servers[0] ])
+    killallServers([ servers[0] ])
 
     await reRunServer(servers[0])
   })

diff --git a/shared/core-utils/plugins/hooks.ts b/shared/core-utils/plugins/hooks.ts
index 60f4130f5b05adac67a858afe76d9fe3d4993376..3d59a7428d492901fd044b2a7bf1561dad24ef1f 100644 (file)
--- a/shared/core-utils/plugins/hooks.ts
+++ b/shared/core-utils/plugins/hooks.ts
@@ -8,7 +8,7 @@ function getHookType (hookName: string) {
   return HookType.STATIC
 }
 
-async function internalRunHook <T>(handler: Function, hookType: HookType, result: T, params: any, onError: (err: Error) => void) {
+async function internalRunHook <T> (handler: Function, hookType: HookType, result: T, params: any, onError: (err: Error) => void) {
   try {
     if (hookType === HookType.FILTER) {
       const p = handler(result, params)