diff --git a/docs/migration/migrate_6_0.asciidoc b/docs/migration/migrate_6_0.asciidoc index b58408f8f5a8..84483742037a 100644 --- a/docs/migration/migrate_6_0.asciidoc +++ b/docs/migration/migrate_6_0.asciidoc @@ -45,3 +45,10 @@ This is no longer the case. Now, only commas are a valid query separator: e.g. ` *Details:* Setting the NODE_ENV environment variable can break Kibana processes in unexpected ways, which is especially unfortunate since it is a common environment variable to have configured on a system, and you wouldn't expect it to break anything in Kibana. Kibana will now effectively ignore NODE_ENV entirely. *Impact:* If you're developing a custom plugin that depends on NODE_ENV, you will need to update it to use a different, custom environment variable. + + +[float] +=== Kibana 4.x configuration names using `_` instead of `.` have been removed +*Details:* In Kibana 4.2, we renamed all configuration names in kibana.yml to use `.` as a separator instead of `_`, though the legacy configurations would still continue to work. In 5.0, we started logging deprecation notices whenever the legacy configurations were encountered. In 6.0 onward, legacy configuration names that use an underscore instead of a dot will no longer work. + +*Impact:* Any usages of underscore separated configuration names in kibana.yml need to be updated to their modern equivalents. See <> for accepted configurations. diff --git a/src/server/config/__tests__/complete.js b/src/server/config/__tests__/complete.js index b5c7da5f557d..668c45587e6d 100644 --- a/src/server/config/__tests__/complete.js +++ b/src/server/config/__tests__/complete.js @@ -73,10 +73,14 @@ describe('server config complete', function () { expect(server.log.called).to.be(false); }); - it('should transform deprecated settings ', function () { + it('should transform server.ssl.cert to server.ssl.certificate', function () { const kbnServer = { settings: { - port: 8080 + server: { + ssl: { + cert: 'path/to/cert' + } + } } }; @@ -88,7 +92,9 @@ describe('server config complete', function () { const config = { get: sinon.stub().returns({ server: { - port: 8080 + ssl: { + certificate: 'path/to/cert' + } } }) }; diff --git a/src/server/config/transform_deprecations.js b/src/server/config/transform_deprecations.js index c4b498971215..23fb18481b66 100644 --- a/src/server/config/transform_deprecations.js +++ b/src/server/config/transform_deprecations.js @@ -15,42 +15,9 @@ const serverSslEnabled = (settings, log) => { const deprecations = [ //server - rename('port' ,'server.port'), - rename('host', 'server.host'), - rename('pid_file', 'pid.file'), - rename('ssl_cert_file', 'server.ssl.certificate'), rename('server.ssl.cert', 'server.ssl.certificate'), - rename('ssl_key_file', 'server.ssl.key'), unused('server.xsrf.token'), serverSslEnabled, - - // logging - rename('log_file', 'logging.dest'), - - // kibana - rename('kibana_index', 'kibana.index'), - rename('default_app_id', 'kibana.defaultAppId'), - - // es - rename('ca', 'elasticsearch.ssl.ca'), - rename('elasticsearch_preserve_host', 'elasticsearch.preserveHost'), - rename('elasticsearch_url', 'elasticsearch.url'), - rename('kibana_elasticsearch_client_crt', 'elasticsearch.ssl.cert'), - rename('kibana_elasticsearch_client_key', 'elasticsearch.ssl.key'), - rename('kibana_elasticsearch_password', 'elasticsearch.password'), - rename('kibana_elasticsearch_username', 'elasticsearch.username'), - rename('ping_timeout', 'elasticsearch.pingTimeout'), - rename('request_timeout', 'elasticsearch.requestTimeout'), - rename('shard_timeout', 'elasticsearch.shardTimeout'), - rename('startup_timeout', 'elasticsearch.startupTimeout'), - rename('verify_ssl', 'elasticsearch.ssl.verify'), - - // tilemap - rename('tilemap_url', 'tilemap.url'), - rename('tilemap_min_zoom', 'tilemap.options.minZoom'), - rename('tilemap_max_zoom', 'tilemap.options.maxZoom'), - rename('tilemap_attribution', 'tilemap.options.attribution'), - rename('tilemap_subdomains', 'tilemap.options.subdomains') ]; export const transformDeprecations = createTransform(deprecations);