codersaur / SmartThings

Samsung SmartThings SmartApps and Device Handlers
Apache License 2.0
300 stars 541 forks source link

influxdb-logger use DNS instead of IP? #18

Open mvillarejo opened 7 years ago

mvillarejo commented 7 years ago

Hello,

first of all, thanks for sharing it really a brilliant repo.

Just a simple question, I've tried to use a DNS instead of an IP for the influxDB logger but it seems it does not work (no exceptions) tried with IP and it worked fine.

Is there any possible changes that can enable DNS resolution on the server name?

Thanks

mvillarejo commented 7 years ago

Does someone knows how this can be implemented? Thanks

jcbize commented 6 years ago

@mvillarejo: See comments on https://github.com/codersaur/SmartThings/issues/16.

You'll need to uncomment the section of code that begins with "For reference, code that could be used for WAN hosts..."

Once you do that, DNS resolution happens automatically.

7c commented 5 years ago

I can confirm that you need to use IP instead of DNS. I could not find out why not but it does not work with hostname instead of IP