Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added the new config since kibana 4.2.x, support for custom log file and server.basePath #79

Merged
merged 2 commits into from
Mar 3, 2016
Merged

Conversation

caskolkm
Copy link

Since kibana 4.2.x they changed the syntax for the config file. An other feature is the server.basePath which allows you to add a root context path. This will fix #78

Support for custom log file and server.basePath
@caskolkm
Copy link
Author

@jlambert121 can you review this?

@jlambert121
Copy link
Owner

This looks great, thanks! Could you add some tests around the version selection?

@caskolkm
Copy link
Author

caskolkm commented Mar 2, 2016

@jlambert121 thnx, i added some tests, hope you like it!

@jlambert121
Copy link
Owner

Thanks! Tests will save your butt some day. :)

jlambert121 added a commit that referenced this pull request Mar 3, 2016
Added the new config since kibana 4.2.x, support for custom log file and server.basePath
@jlambert121 jlambert121 merged commit d6b07fd into jlambert121:master Mar 3, 2016
@caskolkm
Copy link
Author

caskolkm commented Mar 3, 2016

Thnx, can you release a new version of this module?

@jlambert121
Copy link
Owner

2.5.0 pushed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for server.basePath
2 participants