fix(#124): Improve localhost handling, add cache for ipv6 & ipv4 check

[BrandonGillis]

Description

this fix #1114 PR (#124 issue) , by properly implementing localhost, @helloanoop this time I did multiple tests on linux & windows and it works nicely, as a bonus, it also fix [::1] axios issue. Could you check if it's good for you too ?

The cache can create issue though, if you have your localhost listening on ipv6, doing a request with bruno, then changing localhost to ipv4, bruno will still try to reach ipv6, anyway that's a minor issue and a restart of bruno will clear the cache

Contribution Checklist:

• The pull request only addresses one issue or adds one feature.
• The pull request does not introduce any breaking changes
• I have added screenshots or gifs to help explain the change if applicable.
• I have read the contribution guidelines.
• Create an issue and link to the pull request.

[BrandonGillis]

Before merging, I still need to handle the case of doing a dns.lookup first, if localhost have a custom IP inside the hosts file.
Anyway if you have time could you test this change on your machine to see if this is going to the right direction ?

[helloanoop]

Awesome! I confirm that this is working on my machine @BrandonGillis !

[helloanoop]

Before merging, I still need to handle the case of doing a dns.lookup first, if localhost have a custom IP inside the hosts file.

@BrandonGillis Can we do that as a separate PR? I think we can ship this

[BrandonGillis]

Before merging, I still need to handle the case of doing a dns.lookup first, if localhost have a custom IP inside the hosts file.

@BrandonGillis Can we do that as a separate PR? I think we can ship this

No problem for me, we can do that! :)