[Feature request] snmpcollector not able to send data to external influxdb when behind corp. proxy

[fredma]

• What SnmpCollector version are you using?
  0.8.1

• What OS are you running snmpcollector on?

rehl7.8

• What did you do?

We have snmpcollector running in docker container and I defined the env var for HTTP_PROXY and HTTPS_PROXY

        "Env": [
            "HTTP_PROXY=http://myproxy:80/",
            "HTTPS_PROXY=http://myproxy:80/",
            "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
            "PATHS_HOME=/opt/snmpcollector/",
            "SNMPCOL_GENERAL_DATA_DIR=/opt/snmpcollector//data/",
            "SNMPCOL_GENERAL_LOG_DIR=/opt/snmpcollector//log/"
        ],

• What was the expected result?

snmp collector runs inside a corp network and it is configured to send data to external influxdb endpoint (outside of corp network)

The expected result should be a successful connection to external influxdb endpoint.

• What happened instead?

snmpcollector is not able to connect and the below error show in the snmpcollector.log

time="2021-02-22 10:41:23" level=error msg="ERROR on Write batchPoint in DB default (3 points) | elapsed : 30.000279439s | Error: Post https://<EXTERNAL_INFLUXDB_FQDN>:443/write?consistency=&db=snmp&precision=s&rp=autogen: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers) "

To verify this is an issue in snmpcollector, I ran a curl command within the container and it was able to use the proxy defined in HTTPS_PROXY to reach external endpoint e.g. google.co.uk

[fredma]

I believe this is using the influxdb client and a ticket has been raised for this for influxdb
influxdata/telegraf#1588
The fix has already been committed in influxdata/influxdb@c67a054
and possibly snmpcollector would need to use the latest influxdb client as per comments in
influxdata/influxdb#11035

[fredma]

Hi, I managed to fix the issue by applying the following patch on the 0.9.0 version.

http_proxy.patch.txt

Can you please review and let me know if it is good to merge upstream?

Thanks,

Fred

[toni-moreno]

Hello @fredma Could you please, fork the repo and send a PR with these changes please?

[fredma]

Thanks @toni-moreno, I will do that.

[fredma]

Here you go

#474

[fredma]

Sorry, I was on holidays.
Thanks for the merge.