[#271] Avoid multiple client IP addresses by using HTTPS

nchammas / flintrock

A command-line tool for launching Apache Spark clusters.

Apache License 2.0

636 stars 116 forks source link

Closed boechat107 closed 5 years ago

boechat107 commented 5 years ago

This PR makes the following changes:

I tested this PR by calling the service from many different clients using both HTTP and HTTPS protocols.

As suggested in #272, just replacing the protocol seems to avoid the problem of receiving multiple IP addresses from checkip.

Fixes #271

nchammas commented 5 years ago

Thanks for sticking through the research on this issue @boechat107 and settling on this nice, simple solution!