the tcpcheck is always false in meshsync

[jxs1211]

Current Behavior

1. when meshsync controller is checking the broker's service endpoint, the tcpcheck for external endpoint always return false, which make the tcp connection check can't be executed essentially
2. if the endpoint external address is localhost, the TcpCheck will always be executed.

Expected Behavior

1. the external endpoint will be checked successfully
2. the TcpCheck is not necessary to be executed if the external address is localhost

Screenshots/Logs

	// If external endpoint not reachable
	if !utils.TcpCheck(endpoint.External, &utils.MockOptions{}) && endpoint.External.Address != "localhost" {
		newUrl, err = neturl.Parse(url)
		if err != nil {
			return nil
		}

Environment

• Host OS: Mac
• Platform: Kubernetes
• Meshery Server Version: n/a
• Meshery Client Version: n/a

---

Contributor Guides and Resources

• 🛠 Meshery Build & Release Strategy
• 📚 Instructions for contributing to documentation
  • Meshery documentation site and source
• 🎨 Wireframes and designs for Meshery UI in Figma (open invite)
• 🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

[welcome]

Thanks for opening this issue. A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Community Welcome Guide and sure to join the community Slack.

[jxs1211]

@leecalcote I think it's an issue, and I filed a pr for it, pls take a look.