search

graylog-labs / graylog-plugin-metrics-reporter

Graylog Metrics Reporter Plugins
https://www.graylog.org/
GNU General Public License v3.0
80 stars 18 forks source link

Strip '/' from influxdb URI path, it cause error for database name #7

Closed desdulianto closed 8 years ago

desdulianto commented 8 years ago

getPath() from influxdb URI doesn't strip out "/" prefix from the database name and is causing error 404 when graylog tries to write data to influxdb. Fixed this by strips the "/" prefix from database name.

garybot2 commented 8 years ago

@desdulianto: We need you to sign our CLA to be able to merge your pull request. Please click "Details" next to the licence/cla check and sign the CLA you are being shown.

joschi commented 8 years ago

@desdulianto In order to continue, we need you to sign the CLA at https://cla-assistant.io/Graylog2/graylog-plugin-metrics-reporter?pullRequest=7

Thanks!

joschi commented 8 years ago

@desdulianto Thank you very much for your contribution to this project!

I've opted to solve the issue in a slightly different way, see 6f095d971032152d73f53d02ce987a1f04009006.

desdulianto commented 8 years ago

Great! Keep up the good work.

Thank you.