Resolve TODO about MacOS cannot resolve localhost

[PerchunPak]

This bug exists deep inside MacOS, so we can't do anything better, than if statement, that will replace localhost with 127.0.0.1.

BTW, socket.getaddrinfo('localhost', 0) returns ::1 twice and 127.0.0.1 twice too.

[ItsDrike]

The only concern I have here is if someone used localhost to refer to the IPv6 localhost variant. I don't think there's a huge amount of people running servers on IPv6 though, so I'm not sure how major this issue is, and if they do encounter it, we can just tell them to use ::1 instead of localhost.

[PerchunPak]

Related: #268.

[PerchunPak]

The only concern I have here is if someone used localhost to refer to the IPv6 localhost variant. I don't think there's a huge amount of people running servers on IPv6 though, so I'm not sure how major this issue is, and if they do encounter it, we can just tell them to use ::1 instead of localhost.

We can also raise a warning, when replacing localhost.

[ItsDrike]

We can also raise a warning, when replacing localhost.

I like that idea. Should we do it regardless of the platform, or only on macos?

[PerchunPak]

With the current implementation, we should raise a warning always, as user that wants to use IPv6 would be really confused. So we probably should replace localhost only on mac as well as raising the warning only there.

I would also be annoyed to see this warning for no real reason, especially if I don't know about performance issues with localhost.

[ItsDrike]

Well, now we just hope it doesn't break anything somewhere for someone.