-
Notifications
You must be signed in to change notification settings - Fork 6
Description
Was trying to get the library to use HTTPs. Tried various ways, URIs, configurations, specifying different writers and nothing worked. I took a look at the code. I think that the problem is that the InfulxdbHttpWriter, the class that actually does the HTTP posting, has an private URI object. The way this object gets built up doesn't allow anyone to use HTTPs. You cannot specify HTTPs on the InfluxConfig object and the URI overload parses the URI into an InfluxConfig object and then back again into an URI. You lose the fact that the URI wanted or needed HTTPs. My suggestion would be to not telescope to the InfluxConfig overload but rather set the internatl URI to the one passed into the constructor. From a consumer's perspective this is the behaviour I expected and was surprised to see it was different.