Check plugin version on startup

.gitignore

@@ -21,7 +21,7 @@ target
 /esvm
 .htpasswd
 .eslintcache
-plugins
+/plugins/
 data
 disabledPlugins
 webpackstats.json

src/cli_plugin/install/kibana.js

@@ -2,7 +2,7 @@ import _ from 'lodash';
 import { fromRoot } from '../../utils';
 import KbnServer from '../../server/kbn_server';
 import readYamlConfig from '../../cli/serve/read_yaml_config';
-import { versionSatisfies, cleanVersion } from './version';
+import { versionSatisfies, cleanVersion } from '../../utils/version';
 import { statSync } from 'fs';
 
 export function existingInstall(settings, logger) {

src/server/kbn_server.js

@@ -28,6 +28,9 @@ module.exports = class KbnServer {
       // find plugins and set this.plugins
       require('./plugins/scan'),
 
+      // make sure that all plugins expect the current version of Kibana
+      require('./plugins/check_version'),
+
       // tell the config we are done loading plugins
       require('./config/complete'),
 

src/server/plugins/check_version.js

@@ -0,0 +1,36 @@
+import { cleanVersion, versionSatisfies } from '../../utils/version';
+import { get } from 'lodash';
+
+function compatibleWithKibana(kbnServer, plugin) {
+  //core plugins have a version of 'kibana' and are always compatible
+  if (plugin.kibanaVersion === 'kibana') return true;
+
+  const pluginKibanaVersion = cleanVersion(plugin.kibanaVersion);
+  const kibanaVersion = cleanVersion(kbnServer.version);
+
+  return versionSatisfies(pluginKibanaVersion, kibanaVersion);
+}
+
+export default async function (kbnServer, server, config) {
+  //because a plugin pack can contain more than one actual plugin, (for example x-pack)
+  //we make sure that the warning messages are unique
+  const warningMessages = new Set();
+  const plugins = kbnServer.plugins;
+
+  for (let plugin of plugins) {
+    const version = plugin.kibanaVersion;
+    const name = get(plugin, 'pkg.name');
+
+    if (!compatibleWithKibana(kbnServer, plugin)) {
+      const message = `Plugin "${name}" was disabled because it expected Kibana version "${version}", and found "${kbnServer.version}".`;
+      warningMessages.add(message);
+      plugins.delete(plugin);
+    }
+  }
+
+  for (let message of warningMessages) {
+    server.log(['warning'], message);
+  }
+
+  return;
+};

src/server/plugins/plugin.js

@@ -59,6 +59,12 @@ module.exports = class Plugin {
     this.uiExportsSpecs = opts.uiExports || {};
     this.requiredIds = opts.require || [];
     this.version = opts.version || pkg.version;
+
+    // Plugins must specify their version, and by default that version should match
+    // the version of kibana down to the patch level. If these two versions need
+    // to diverge, they can specify a kibana.version in the package to indicate the
+    // version of kibana the plugin is intended to work with.
+    this.kibanaVersion = opts.kibanaVersion || _.get(pkg, 'kibana.version', this.version);
     this.externalPreInit = opts.preInit || _.noop;
     this.externalInit = opts.init || _.noop;
     this.configPrefix = opts.configPrefix || this.id;
@@ -89,17 +95,14 @@ module.exports = class Plugin {
     };
   }
 
-  async readConfig() {
+  async readConfigSchema() {
     let schema = await this.getConfigSchema(Joi);
-    let { config } = this.kbnServer;
-    config.extendSchema(this.configPrefix, schema || defaultConfigSchema);
-
-    if (config.get([...toPath(this.configPrefix), 'enabled'])) {
-      return true;
-    } else {
-      config.removeSchema(this.configPrefix);
-      return false;
-    }
+    return schema || defaultConfigSchema;
+  }
+
+  get enabled() {
+    const { config } = this.kbnServer;
+    return config.get([...toPath(this.configPrefix), 'enabled']);
   }
 
   async preInit() {

src/server/plugins/plugin_collection.js

@@ -2,11 +2,23 @@
 import PluginApi from './plugin_api';
 import { inspect } from 'util';
 import { get, indexBy } from 'lodash';
+import toPath from 'lodash/internal/toPath';
 import Collection from '../../utils/collection';
 
 let byIdCache = Symbol('byIdCache');
 let pluginApis = Symbol('pluginApis');
 
+async function addPluginConfig(pluginCollection, plugin) {
+  const configSchema = await plugin.readConfigSchema();
+  let { config } = pluginCollection.kbnServer;
+  config.extendSchema(plugin.configPrefix, configSchema);
+}
+
+function removePluginConfig(pluginCollection, plugin) {
+  let { config } = pluginCollection.kbnServer;
+  config.removeSchema(plugin.configPrefix);
+}
+
 module.exports = class Plugins extends Collection {
 
   constructor(kbnServer) {
@@ -27,21 +39,26 @@ module.exports = class Plugins extends Collection {
     // clear the byIdCache
     this[byIdCache] = null;
 
-    for (let product of output) {
-
-      if (product instanceof api.Plugin) {
-        let plugin = product;
-        this.add(plugin);
-
-        let enabled = await plugin.readConfig();
-        if (!enabled) this.delete(plugin);
-        continue;
+    for (let plugin of output) {
+      if (!plugin instanceof api.Plugin) {
+        throw new TypeError('unexpected plugin export ' + inspect(plugin));
       }
 
-      throw new TypeError('unexpected plugin export ' + inspect(product));
+      await this.add(plugin);
+      if (!plugin.enabled) this.delete(plugin);
     }
   }
 
+  async add(plugin) {
+    await addPluginConfig(this, plugin);
+    super.add(plugin);
+  }
+
+  delete(plugin) {
+    removePluginConfig(this, plugin);
+    super.delete(plugin);
+  }
+
   get byId() {
     return this[byIdCache] || (this[byIdCache] = indexBy([...this], 'id'));
   }

src/ui/__tests__/fixtures/plugin_async_foo/package.json

@@ -1,4 +1,4 @@
 {
   "name": "plugin_async_foo",
-  "version": "0.0.0"
+  "version": "kibana"
 }

src/ui/__tests__/fixtures/plugin_bar/package.json

@@ -1,4 +1,4 @@
 {
   "name": "plugin_bar",
-  "version": "0.0.0"
+  "version": "kibana"
 }

src/ui/__tests__/fixtures/plugin_foo/package.json

@@ -1,4 +1,4 @@
 {
   "name": "plugin_foo",
-  "version": "0.0.0"
+  "version": "kibana"
 }