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

better handling for invalid driver paths #369

Closed
kares opened this issue Jan 21, 2020 · 0 comments · Fixed by logstash-plugins/logstash-integration-jdbc#15 · May be fixed by #370
Closed

better handling for invalid driver paths #369

kares opened this issue Jan 21, 2020 · 0 comments · Fixed by logstash-plugins/logstash-integration-jdbc#15 · May be fixed by #370

Comments

@kares
Copy link

kares commented Jan 21, 2020

currently jdbc_driver_path loading is delayed until the first statement is executed,
this means that if a plugin is using schedule a failure in driver loading (such as insufficient read access) will not be visible in the plugin log but only in console.

this is confusing esp. when using LS as a service (console not visible unless journald is inspected) - while LS is self contained its bringing in a driver.jar via this plugin and its not very clear permissions should be taken into account.

plugin should potentially document this + improve its driver.jar checking for read access ...

kares added a commit to kares/logstash-input-jdbc that referenced this issue Jan 21, 2020
kares added a commit to logstash-plugins/logstash-integration-jdbc that referenced this issue Jan 22, 2020
these are non-critical improvements for (input) driver loading.

issues are kept on the original repo since they concern the plugin.
this is expected to resolve logstash-plugins/logstash-input-jdbc#369
(also partially relates to logstash-plugins/logstash-input-jdbc#362)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant