Print Page - UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1

Support => TNet 3 Support => Topic started by: pyscho2day on December 10, 2013, 11:57:09 PM

Title: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: pyscho2day on December 10, 2013, 11:57:09 PM

This is not happening all of the time and it hasn't happened in while. I have not modified any of your code that would control that what so ever. Would it be possible to get a check in the logic that if for some reason it get 127.0.0.1 when it tries to open the ports to try again? Or maybe to kick out an exception.

I currently check that the ports open and I don't get an error, but this is just as bad as an error and so far i have not found a way with out modifying your code to find out what ports were opened.

Not in a rush but if you could put this on your radar it would be appreciated. Also if anyone else has had this issue post here. if no one then maybe I am just the lucky one.

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: ArenMook on December 11, 2013, 02:17:23 AM

I'm not quite clear on this one... what do you mean? 127.0.0.1 is a loopback address pointing to your machine, so I am not quite sure what it has to do with open ports.

Do you mean that it doesn't detect the correct IP sometimes and sends that info to the lobby server?

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: pyscho2day on December 11, 2013, 10:13:03 AM

Actually the lobby server just gets 2 entries for the local ip, not the local ip and the outside ip. The issue is that the ports being opened on the router get the 127.0.0.1 ip address assigned to them and not the local ip 10.0.0.### that it should. When this happens the router does not know where to send the incoming traffic.

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: Rogdor on December 11, 2013, 05:55:36 PM

I'm not sure if this is exactly related:

For me, the TNet server sets the external IP incorrectly, as the same as my internal. This happens about a third of the time when I launch it. TNet.Tools.externalAddress also randomly grabs the internal address sometimes.

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: pyscho2day on December 11, 2013, 11:17:21 PM

I did some testing to night and every time the upnp ports were opened with the localhost ip the ips that were reported to the lobby server were both the private ip not the public ip.

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: ArenMook on December 12, 2013, 06:27:34 AM

The external IP detection method involves querying a remote website, so it may not always work. Ideally you would put the query script on your own server instead. I did that with starlink. It's a short PHP script that I've posted before.

<?php
echo 'Current IP Address: ' . $_SERVER['REMOTE_ADDR'];
?>

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: pyscho2day on December 12, 2013, 11:05:40 AM

I do use the detection on an external website, and if it was not reachable that would be fine. I am more concerned with the ports being opened on 127.0.0.1. I made a small script copying what you do in the TNTools.cs to obtain the local ip and verify it. I let this run for 4 hours and not once did i get 127.0.0.1 as the local (lan) ip. I then tried to start a server and the first time it used 127.0.0.1 as the ip to open the ports on. I am trying to avoid modifying your code as when we get an update it gets wiped out. I thought about inheriting the TNet.UPnP but there are protected members that are not accessible (one being the local ip). I was going to do that so i could check the ip and if it was the 127.0.0.1 the check again to make sure i get the correct local ip.

I had put a fix in version 1.7.x on mine in the past but have updated since then. What I did was get the local ip and put that into a variable and then checked it, if it was 127.0.0.1 i would then get and check again.

I believe this is what I had added. (* in front of now or edited lines)

void Open (int port, bool tcp, OnPortRequest callback)
{
  int id = (port << 8) | (tcp ? 1 : 0);
 
  if (port > 0 && !mPorts.Contains(id) && mStatus != Status.Failure)
  {
    mPorts.Add(id);
*   IPAddress tempLocal = Tools.localAddress;
*   for (int i = 0; i < 10; i++)
*   {
*     if (tempLocal.ToString() != "127.0.0.1")
*     {
*       return;
*     }
*     else
*     {
*       tempLocal = Tools.localAddress;
*     }
*   }
 
    ExtraParams xp = new ExtraParams();
    xp.callback = callback;
    xp.port = port;
    xp.protocol = tcp ? ProtocolType.Tcp : ProtocolType.Udp;
    xp.action = "AddPortMapping";
    xp.request = "<NewRemoteHost></NewRemoteHost>\n" +
      "<NewExternalPort>" + port + "</NewExternalPort>\n" +
      "<NewProtocol>" + (tcp ? "TCP" : "UDP") + "</NewProtocol>\n" +
      "<NewInternalPort>" + port + "</NewInternalPort>\n" +
*     "<NewInternalClient>" + tempLocal + "</NewInternalClient>\n" +
      "<NewEnabled>1</NewEnabled>\n" +
      "<NewPortMappingDescription>" + name + "</NewPortMappingDescription>\n" +
      "<NewLeaseDuration>0</NewLeaseDuration>\n";
 
    xp.th = new Thread(OpenRequest);
    lock (mThreads) mThreads.Add(xp.th);
    xp.th.Start(xp);
  }
  else if (callback != null)
  {
    callback(this, port, tcp ? ProtocolType.Tcp : ProtocolType.Udp, false);
  }
}
 

I am at work right now so I can not verify, but if you would like I can try this when I get home and make sure that 127.0.0.1 is no longer being used.

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: ArenMook on December 12, 2013, 11:10:51 AM

Certainly, and keep in mind you can just email me your changes (provided you used the latest version which is currently 1.8.4).

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: pyscho2day on February 03, 2014, 11:47:46 AM

Aren,

I have been running this fix for a month now and have no seen the 127.0.0.1 show up when opening ports.

Note: Lines with a * are added/modified.

        void Open (int port, bool tcp, OnPortRequest callback)
        {
                int id = (port << 8) | (tcp ? 1 : 0);
 
                if (port > 0 && !mPorts.Contains(id) && mStatus != Status.Failure)
                {
                        mPorts.Add(id);
 
*                       IPAddress tempLocal = Tools.localAddress;
*                       int tempCnt = 0;
*                       while (tempLocal.ToString() == "127.0.0.1" && tempCnt < 20)
*                       {
*                               tempCnt++;
*                               tempLocal = Tools.localAddress;
*                               Thread.Sleep(10);
*                       }
 
 
                        ExtraParams xp = new ExtraParams();
                        xp.callback = callback;
                        xp.port = port;
                        xp.protocol = tcp ? ProtocolType.Tcp : ProtocolType.Udp;
                        xp.action = "AddPortMapping";
                        xp.request = "<NewRemoteHost></NewRemoteHost>\n" +
                                "<NewExternalPort>" + port + "</NewExternalPort>\n" +
                                "<NewProtocol>" + (tcp ? "TCP" : "UDP") + "</NewProtocol>\n" +
                                "<NewInternalPort>" + port + "</NewInternalPort>\n" +
*                               "<NewInternalClient>" + tempLocal + "</NewInternalClient>\n" +
                                "<NewEnabled>1</NewEnabled>\n" +
                                "<NewPortMappingDescription>" + name + "</NewPortMappingDescription>\n" +
                                "<NewLeaseDuration>0</NewLeaseDuration>\n";
 
                        xp.th = new Thread(OpenRequest);
                        lock (mThreads) mThreads.Add(xp.th);
                        xp.th.Start(xp);
                }
                else if (callback != null)
                {
                        callback(this, port, tcp ? ProtocolType.Tcp : ProtocolType.Udp, false);
                }
        }

edit: Added a couple line to exit the while loop if a suitable ip can not be obtained. Thanks to @GantZ_Yaka for letting me know that he has a laptop that is never able to get the local ip and always returns the 127.0.0.1

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: ArenMook on February 04, 2014, 03:59:33 AM

I've actually modified TNTools.localAddress to use GetHostAddresses instead in the Pro version. I wonder if this would also resolve the DNS issue?

        static public IPAddress localAddress
        {
                get
                {
                        if (mLocalAddress == null)
                        {
#if UNITY_IPHONE
                                NetworkInterface[] nis = NetworkInterface.GetAllNetworkInterfaces();
 
                                foreach (NetworkInterface ni in nis)
                                {
                                        IPInterfaceProperties IPInterfaceProperties = ni.GetIPProperties();
                                        UnicastIPAddressInformationCollection UnicastIPAddressInformationCollection = IPInterfaceProperties.UnicastAddresses;
 
                                        foreach (UnicastIPAddressInformation UnicastIPAddressInformation in UnicastIPAddressInformationCollection)
                                        {
                                                if (UnicastIPAddressInformation.Address.AddressFamily == AddressFamily.InterNetwork)
                                                {
                                                        mLocalAddress = UnicastIPAddressInformation.Address;
                                                        break;
                                                }
                                        }
                                }
#else
                                try
                                {
                                        IPAddress[] ips = Dns.GetHostAddresses(Dns.GetHostName());
 
                                        for (int i = 0; i < ips.Length; ++i)
                                        {
                                                if (IsValidAddress(ips[i]))
                                                {
                                                        mLocalAddress = ips[i];
                                                        break;
                                                }
                                        }
 
                                        //IPHostEntry ent = Dns.GetHostEntry(Dns.GetHostName());
 
                                        //foreach (IPAddress ip in ent.AddressList)
                                        //{
                                        //    if (IsValidAddress(ip))
                                        //    {
                                        //        mLocalAddress = ip;
                                        //        break;
                                        //    }
                                        //}
                                }
#if DEBUG
                                catch (System.Exception ex)
                                {
                                        System.Console.WriteLine("TNTools.LocalAddress: " + ex.Message);
                                        mLocalAddress = IPAddress.Loopback;
                                }
#else
                                catch (System.Exception) { mLocalAddress = IPAddress.Loopback; }
#endif // DEBUG
#endif // UNITY_IPHONE
                        }
                        return mLocalAddress;
                }
        }

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: pyscho2day on February 04, 2014, 10:40:48 AM

That is possible. Unfortunatly I don't have the pro license yet. Once I build my new pc I will probably upgrade as its only 30$ more.

Side note, @GantZ_Yaka was having issues with one of his machines always returning with 127.0.0.1 and here is what he found. (he was messaging me directly)

Quote from: GantZ_Yaka

Windows 7, I found the reason due to which the IP was always taken 127.0.0.1.
Fault location:
IPHostEntry iphostentry = Dns.GetHostByName(strHostName);
Im from russia and in the name of the computer (My computer - Properties) except the letters were also Russian letters.
When iphostentry is taken not only of the Latin letters, the program itself could not find the local IP using that Hostname.
When I changed the network name in Latin letters, everything was working correctly

Title: Re: UPnP.OpenTCP and UPnP.OpenUDP are opening port on 127.0.0.1
Post by: ArenMook on February 05, 2014, 04:46:57 PM

You don't need the Pro license, I posted the code I changed.

Tasharen Entertainment Forum

Support => TNet 3 Support => Topic started by: pyscho2day on December 10, 2013, 11:57:09 PM