samitpal / influxdb-router Goto Github PK

During start up influxdb-router does not log anything about unhealthy back ends (InfluxDB hosts). It can lead to confusion.

Dear Samit,

I have looked at your router and I'm trying to figure out if this would allow us to use it for a specific use-case.

We collect stats for a whole range of customers centrally influx but have the option give customers access to their own data using grafana. We want to segregate data to give customers access to their own data AND still have access to all data from an admin view-point.

I'm correct that your influxdb-router, would allow the option define as a "in-between" gateway/proxy that acts as a "filter" while inserting and/or fetching influx data?

Example:

• "Customer X" has access to influxdata with tags "customer x"?
• "Customer Y" has access to influxdata with tags "customer x" & "customer y"
• "Admin" has access to all influxdata

• Modify the http proxy port from 80 to 8090
• Modify the https proxy port from 443 to 8443

There should be a version flag which outs the version of the binary in use.

Mask

• api key, creds during startup.
• api key logged for every request processed successfully.

Change needs to go to line https://github.com/samitpal/influxdb-router/blob/master/Makefile#L16

Some kind of throttling per customer perhaps leveraging the /debug/requests InfluxDB end point

Use the service name instead of the api key to construct internal InfluxDB-router metrics.

i.e

influx_router.<api_key>.outgoing_queue...

should be replaced by

influx_router.<service_name>.outgoing_queue...

Please update Makefile to use the lastest version
INFLUXDB-ROUTER_VERSION := 0.1.2

change to
INFLUXDB-ROUTER_VERSION := 0.1.3

Influxdb-router can make a call with the configured auth creds to the backends and verify that the creds work as expected. This is primarily for better troubleshooting etc.

Perhaps using goreleaser itself
Doc: https://goreleaser.com/docker/

Integrate with travis CI

In other words when somebody types
$influxdb-router -version
the version, date, commit should be displayed. Right now the version and buildtime gets displayed but only if some one builds the binary using the make command

If the backend InfluxDB is configured for https communication, the influxDB router needs to work as expected. Verify the same

Some times a good random string is likely to have the !,- or * characters. Remove the restriction as indicated at https://github.com/samitpal/influxdb-router/blob/master/config.toml#L4

The restriction was purely because of us using the api key as as an env variable when using auth from env. linux does not allow shell variables containing those characters.