Previous code passed the index parameter from lodash.map to
readFileSync, so it was essentially the same thing as:
```
map(config.get('server.ssl.certificateAuthorities'), (file, index) => {
return readFileSync(file, index);
})
```
The upgrade to node8 seems to change the behavior and this was causing
an error:
```
FATAL TypeError: "options" must be a string or an object, got number instead.
at getOptions (fs.js:75:11)
at fs.readFileSync (fs.js:549:13)
at server.connection._extends.tls.ca.config.get.map (/Users/tsullivan/code/kibana/src/server/http/setup_connection.js:43:76)
at Array.map (<anonymous>)
at KbnServer.exports.default (/Users/tsullivan/code/kibana/src/server/http/setup_connection.js:43:59)
at KbnServer.mixin (/Users/tsullivan/code/kibana/src/server/kbn_server.js:97:16)
at KbnServer.exports.default (/Users/tsullivan/code/kibana/src/server/http/index.js:20:19)
at KbnServer.mixin (/Users/tsullivan/code/kibana/src/server/kbn_server.js:97:16)
at <anonymous>
at process._tickCallback (internal/process/next_tick.js:188:7)
at Function.Module.runMain (module.js:686:11)
at startup (bootstrap_node.js:187:16)
at bootstrap_node.js:608:3
optimizer crashed with status code 1
```
This PR fixes the issue by making the mapping through the
certificateAuthorities files with a more explicit handling of the map
items, as well as remove the unnecessary lodash import
* add failing test that would have caught this situation
* Implement onEmbeddableConfigChanged to pick up url changes from soft refreshes
* Add tests for legend and pie slice color as well as removing uiState keys to ensure it resets the config values
* address removing state from url as well as setting it.
* Fix up tests and add a link to the legend issue
* only emit once
* use for in, use better mocked clearAllKeys function
* Remove switch to setSilent
* Adjust test so it fails because of bug found where two charts will both have their slice colors updated
* Embeddable should only listen for changes to their panel's config state
* Default embeddable config to an empty object when creating a new embeddable
This PR evolves Kuery into a simplified, more lucene-like language. In a follow up PR we plan on getting rid of the language switcher dropdown and instead add a checkbox that will allow users to opt-in to this syntax, which will feel less like a separate language and more like a set of enhancements to the existing lucene query syntax.
This PR also updates the simple grammar to solve the following:
* Wildcards in field names
* "is one of" shorthand (e.g. foo:(bar OR baz OR qux))
* Don't split on whitespace (require explicit AND/ORs, case insensitive)
* "exists" query (e.g. foo:*)
* Improved range query shorthands (e.g. foo <= 1)
* Wildcard queries
Since this new syntax is simpler and does not support every filter type like old Kuery did, this PR also brings back the filter bar when kuery is selected in the language picker.
See the documentation updates in this PR if you need an overview of the new syntax (and let us know if the documentation is lacking).
* Increase test runs to reliably trigger failure
* Determine where dashboard is failing to pop into view mode
* Drop down to only one check
* Go back to only one run of state tests
* First stab at refactoring typeahead
* Don't double submit on enter
* Add item templating
* Remove unnecessary test
* Don't submit on enter
* Add clarifying comment
* Fix history log key
* Fix key handling
* Don't always hide on submit
* Check items exists
* Simplify select next/prev and reset selected on hide/backspace
* Add test
* Put persistedLog on scope so it can be tested
* Update typeahead items when language changes
* Finish that work I didn't do in the last commit
* Order matters
* use default index pattern when filter-editor is not passed any index patterns
* add functional test
* pass indexPatterns to filterBar directive
* handle check for [undefined]
* use _.compact to test if index patterns are provided
* simplify areIndexPatternsProvided
* Only update status when required
* Add missing Status.PARAMS
* Add requiresUpdateStatus to Vega vis
* Add requiresUpdateStatus for editors
* Fix stupid issue
* Create unit tests for getUpdateStatus method
* [kbn-plugin-helpers] allow opt-ing out of dependency installation
* [plugin-helpers] rename buildIgnoreDependencies to skipInstallDependencies
* [plugin-helpers] use noop3 for test
* [plugin-helpers] fix test description
* [plugin-helpers] fix assertion
* [server/rewriteBasePath] add option to enable basePath rewriting
* [server/rewriteBasePath/docs] end sentences with periods
* [server/rewriteBasePath] simplify Joi schema a smidge
* [server/rewriteBasePath] rename test file to match source
* [server/rewriteBasePath] initialize server in before/after hooks
* [server/rewriteBasePath] rephrase deprecation warning
* [server/config/schema] verify that non-strings are not accepted for basePath
* [server/config/schema] toss a trailing comma in there
