Add more public IP checks for better resilience

Question

Add more public IP checks for better resilience

silverwind opened this issue 5 years ago · 10 comments

See sindresorhus/is-online#64. One or two alternative services to be checked in parallel would probably be beneficial for better resilience against outages.

IssueHunt Summary

adityapatadia has been rewarded.

Backers (Total: $40.00)

issuehunt ($40.00)

Submitted pull Requests

#33 added backup service ipify in case primary service goes down

Tips

Checkout the Issuehunt explorer to discover more funded issues.
Need some help from other developers? Add your repositories on IssueHunt to raise funds.

IssueHunt has been backed by the following sponsors. Become a sponsor

Answer 1 · 2019-06-28T06:35:05.000Z

Yes, it could also check https://www.ipify.org/ through https://github.com/sindresorhus/ipify.

There's also https://checkip.amazonaws.com/

Any other good free trustable IP check APIs? Maybe from Google, Cloudflare, etc? The person taking on this issue should do some research.

Answer 2 · 2019-06-28T06:53:51.000Z

@issuehunt has funded $40.00 to this issue.

Submit pull request via IssueHunt to receive this reward.
Want to contribute? Chip in to this issue via IssueHunt.
Checkout the IssueHunt Issue Explorer to see more funded issues.
Need help from developers? Add your repository on IssueHunt to raise funds.

Answer 3 · 2019-07-12T06:30:23.000Z

ipify seems the best. I will go ahead with it. Another option I saw was https://ip.seeip.org

Answer 4 · 2019-07-12T06:32:46.000Z

Amazon one is not officially documented so I think we should avoid that one.

Answer 5 · 2019-07-12T06:42:57.000Z

@sindresorhus seeing this ipify module, I can't help but think it's very redundant to this module. Maybe deprecate it in favor of this module and include the ipify check here?

Answer 6 · 2019-07-12T06:44:50.000Z

I am not going to use the node module. I will use GOT to get address from their API.

Answer 7 · 2019-07-12T07:10:57.000Z

seeing this ipify module, I can't help but think it's very redundant to this module. Maybe deprecate it in favor of this module and include the ipify check here?

I don't see the point in deprecating it. It can be useful when you don't care about redundancy, or as an easy add when you just want the IP without having to think about how to talk to the API or what HTTP lib to use.

Answer 8 · 2019-07-15T10:45:53.000Z

Opened #33

Answer 9 · 2019-07-23T08:22:35.000Z

Any update on this? Is the PR going to be merged or is it blocked for some reason?

Answer 10 · 2019-08-14T12:46:41.000Z

@sindresorhus has rewarded $36.00 to @adityapatadia. See it on IssueHunt

💰 Total deposit: $40.00
🎉 Repository reward(0%): $0.00
🔧 Service fee(10%): $4.00