Method Not Found From MinecraftServer.getPort()

[jredfox]

	@Override
	public void execute(MinecraftServer server, ICommandSender sender, String[] args) throws CommandException 
	{
		if(!(sender instanceof EntityPlayerMP))
			return;
		try
		{
			server.getServerPort();
//			EntityUtil.sendClipBoard(EnumChatFormatting.WHITE, EnumChatFormatting.BLUE + EnumChatFormatting.UNDERLINE, (EntityPlayerMP)sender, "ServerIP:", ip);
		}
		catch(Exception e)
		{
			e.printStackTrace();
		}
	}

I have my command here for getting the server ip and port. I cannot find where the ip is stored and MinecraftServer.getPort() throws method not found exception in 1.12.2 even though lan server is open. So what just use reflection then to get the port and what about the ip? Use an external website to get the server's public ip adress?

Edited June 25, 2018 by jredfox

[DaemonUmbra]

4 hours ago, jredfox said:

even though lan server is open

Does this mean you're running a physical client? Because getServerPort() is physical server only.

[jredfox]

2 hours ago, DaemonUmbra said:

Does this mean you're running a physical client? Because getServerPort() is physical server only.

no It's a server only command the thing is I think @sideOnly server makes only only usable on dedicated the world.isRemote returned false when using the command and the entity player was mp

Edited June 25, 2018 by jredfox

[Animefan8888]

3 hours ago, jredfox said:

no It's a server only command the thing is I think @sideOnly server makes only only usable on dedicated the world.isRemote returned false when using the command and the entity player was mp

Well first off, why are you trying to get the ip and port?

Edited June 25, 2018 by Animefan8888
Typo

[jredfox]

6 hours ago, Animefan8888 said:

Well first off, why are you trying to get the ip and port?

so people don't have to relog find their ip and port control a control x then give it to other people. Also for people who are lazy aka server owners and don't feel like going to ipchicken.com and finding the port they hosted on

Edited June 25, 2018 by jredfox

[jredfox]

6 hours ago, diesieben07 said:

If you register the command on the dedicated server only (FMLStateEvent#getSide will tell you), this should work. Show how you register the command.

but, I don't want dedicated server I want it working on lan servers to. 

So I got some code in my proxy to get it working for lan servers but, I am unsure if this is proper way to get it????
 

	public static String getPort() {
		IntegratedServer server = Minecraft.getMinecraft().getIntegratedServer();
		ThreadLanServerPing ping = (ThreadLanServerPing) ReflectionUtil.getObject(server, IntegratedServer.class, LanFeildsClient.lanServerPing);
		if(ping == null)
			return "-1";
		return (String)ReflectionUtil.getObject(ping, ThreadLanServerPing.class, LanFeildsClient.address);
	}

and on dedicated server side:
 

	public static String getServerPort(MinecraftServer server) {
		return "" + server.getServerPort();
	}

putting it together:
 

String port = MainJava.isClient ? ClientProxy.getPort() : ServerProxy.getServerPort(server);

Now I still am hard coding the ip to dynamically get it from a website rather the the mc servers cached version. Where is the mc servers cached ip for both lan and dedicated

[jredfox]

3 minutes ago, diesieben07 said:

?‍♂️?‍♂️?‍♂️?‍♂️

 

Just use reflection on MinecraftServer#serverPort, no need to have two separate code-paths.

 

What "cache"? A server does not necessarily know it's own IP.

the server port returns -1 if it's not dedicated. A server should know it's public ip I guess I have the right code then maybe I could do a cached variables so it's not getting it every time online I just won't store it to the disk.

I expected the server to cache it's own public ip in a variable somewhere

My question is for lan servers where is the port stored at properly or did I already do it right?

Edited June 25, 2018 by jredfox

[jredfox]

7 minutes ago, diesieben07 said:

Why do you think that?

 

You... what?

store the ip once gotten once to the right side and keep it there till re-launch since ips can and will change

[Draco18s]

10 minutes ago, jredfox said:

A server should know it's public ip

No, a computational device only knows it's local IP address. It don't know the external IP address of every router and other system it sits behind.

In fact, it can't know its public IP without asking an external source, "Hey, what IP did this message come from?" And even then, if the external source is not properly chosen (e.g. another device on the same internal network), it still doesn't know its "public" IP.

[jredfox]

Just now, Draco18s said:

No, a computational device only knows it's local IP address. It don't know the external IP address of every router and other system it sits behind.

In fact, it can't know its public IP without asking an external source, "Hey, what IP did this message come from?" And even then, if the external source is not properly chosen (e.g. another device on the same internal network), it still doesn't know its "public" IP.

Look at here:

	public static String getPublicIp() throws IOException 
	{
		URL whatismyip = new URL("http://checkip.amazonaws.com");
		BufferedReader in = new BufferedReader(new InputStreamReader(whatismyip.openStream()));
		String ip = in.readLine(); //you get the IP as a String
		return ip;
	}

	public static String getIpv4() throws UnknownHostException 
	{
        InetAddress inetAddress = InetAddress.getLocalHost();
		return inetAddress.getHostAddress();
	}

For this command it's going to use public ip in my JavaUtil.class

[Draco18s]

Yes, that is asking an external device to reply to you.

[Cadiboo]

Before doing all this you should probably look at how IP works and also how ports & port forwarding works.

There are quite a few nuances with this stuff, don’t forget that the internet is a massive mess and pretty much everything on it is jerry rigged.

- Your IP could be localhost (AKA ::1 AKA 127.0.0.1) on single player or 192.168.0.X on LAN or your actual public IP in multiplayer. It all depends on who’s looking at the IP and from where.

Also for the ports, your router could be listening on port 80 for incoming connections and forwarding it to port 500 on your machine which is forwarding it to port 25565 for Minecraft. Just saying that I think your going about this the wrong way, if I were doing it I’d hook into the initial handshakes with new clients and store information about them (IP they connected from, IP & Port they connected to = Sever IP & Port) in an Array and access that array whenever the data is needed by the command. You’d also probably want multiple commands too see local IP and Public IP for the server. Just my two cents hope it helps

[jredfox]

On 6/27/2018 at 9:18 AM, Cadiboo said:

Before doing all this you should probably look at how IP works and also how ports & port forwarding works.

There are quite a few nuances with this stuff, don’t forget that the internet is a massive mess and pretty much everything on it is jerry rigged.

- Your IP could be localhost (AKA ::1 AKA 127.0.0.1) on single player or 192.168.0.X on LAN or your actual public IP in multiplayer. It all depends on who’s looking at the IP and from where.

Also for the ports, your router could be listening on port 80 for incoming connections and forwarding it to port 500 on your machine which is forwarding it to port 25565 for Minecraft. Just saying that I think your going about this the wrong way, if I were doing it I’d hook into the initial handshakes with new clients and store information about them (IP they connected from, IP & Port they connected to = Sever IP & Port) in an Array and access that array whenever the data is needed by the command. You’d also probably want multiple commands too see local IP and Public IP for the server. Just my two cents hope it helps

I did take a look at exactly where the sockets are instantiated on client it's a ping thing that's client only and on server it's like 3 separate threads. The threads are to keep it up on lan side the actual port forwarding process is unclear as nobody seems to know how to make upnp work except for one library I recently found and had to modify it a bit(currently still unoptimzied takes 5 whole seconds to port forward):
https://github.com/adolfintel/WaifUPnP

On Share to Lan: Port forward if applicable on gui or command only

On Server Start: Port forward if applicable on dedicated only

Port forwarding is just telling your internet box it's ok to put your port through the firewall the sockets already make it connection themselves it's you just need to tell the box to open up a port entry

Edited July 3, 2018 by jredfox

[jredfox]

8 hours ago, diesieben07 said:

Port forwarding is more about NAT than about firewalls. But anyways.

5 seconds for port-forwarding is not "unoptimized". Also, this whole endeavour sounds like a bad idea. Don't just automatically muck about with a users router settings. If they know what they are doing, they will do it themselves. If they do not know what they are doing, they should not be modifying their router settings.

5 seconds is highly unoptimized utorrent can do it in less then a second but, again it's an unsafe program to use so I found a safe one and am working on making it mc compatible.

 

It is a good idea for people who want an actual server and cannot mess with the internet box aka anyone < 18 which is who plays minecraft. I looked at the code all it does it add an entry it takes too long due to it calling wait for stuff on pausing the thread way too much rather then just once. I am going to modify it later on my fork:
https://github.com/jredfox/WaifUPnP

Edited July 4, 2018 by jredfox

[Jaideep Heer]

If you want to know what port the LAN server is hosted on, check out HttpUtil#getSuitableLanPort() because that is the class that is called while deciding a port for LAN.

You can also use a ClassTransformer to inject your custom code that can either edit or just note down the port number returned by that function.

 

Check out my ClassTransformer here .

[DaemonUmbra]

1 hour ago, Jaideep Heer said:

check out HttpUtil#getSuitableLanPort()

Pretty sure that would get you a completely new port as the previously suitable port is now bound.

 

If I may speak frankly if someone wants to run a server with friends they can use software like Hamachi to accomplish this task without altering the settings on their router or modem and this sounds to me like one of the things where if users can't put in the effort to learn and configure these things on their own (or in the case of <18 year olds as you put it jred, get permission to do so) then maybe they shouldn't be opening their computers to external connections.

[Jaideep Heer]

9 minutes ago, DaemonUmbra said:

Pretty sure that would get you a completely new port as the previously suitable port is now bound.

 

If you use a ClassTransformer to add an instruction that stores the returned port every time that function is called, it'll tell you the port that is being used currently.

I use this method to make it send a custom port number thereby making that setting configurable.

[jredfox]

4 hours ago, DaemonUmbra said:

Pretty sure that would get you a completely new port as the previously suitable port is now bound.

 

If I may speak frankly if someone wants to run a server with friends they can use software like Hamachi to accomplish this task without altering the settings on their router or modem and this sounds to me like one of the things where if users can't put in the effort to learn and configure these things on their own (or in the case of <18 year olds as you put it jred, get permission to do so) then maybe they shouldn't be opening their computers to external connections.

hamichi is getting into the dangerzone and it doesn't support all of your data you need severe block lag. Utorrent is also getting into the danger zone it's a torrenting malicious program thus you cannot safely use it to port forward.

Also if you read my post people < 18 usually can't mess with the internet box itself with my mod you don't have to just opens up your port just like xbox does. So yeah I don't even trust myself to mess with my box just use UPNP to add a port entry and close when the game is done and if close ports is configed(remove entry from router). The difference is the port entry will be there on the box still just will get no reply until minecraft is open to lan. I wouldn't recommend the config option unless your server is dedicated and you have changed your port. I would also tell people to disable port forwarding if they want to throw the mod in one of those virtual dedicated servers so it doesn't break them

I use a java library it's not that sophisticated so if someone knows something that isn't cling(requires android and looks unsafe as well) then I would like to hear about it. I found one it works on my internet box I can't verify if it works with all routers supporting UPNP.

 

I hear NAT-PMP (or something like that) is a protocol I should also support but, I can't find a java library that does support it and works. I also Noticed PCP or something like that newer routers are supporting. So please help me find a more updated library that supports all of them.

Currently I only Support UPNP with (TCP and UDP) I am not sure how to support NAT-PMP or any newer protocols to make 99% routers compatible not just mine.

Edited July 4, 2018 by jredfox

[jredfox]

25 minutes ago, Jaideep Heer said:

If you use a ClassTransformer to add an instruction that stores the returned port every time that function is called, it'll tell you the port that is being used currently.

I use this method to make it send a custom port number thereby making that setting configurable.

No need for ASM changing the port when opening to lan. Changing the port isn't the issue it's port forwarding telling the internet box hey TCP allow this port through the firewall right after minecraft says hey open this to lan on a server socket.

Yes I also removed /publish command as it was completely broken. I close the lan and re-open it with my command openToLan or openToInternet. Of course removing anything now in days or editing anything requires alot of reflection

You seem to know ASM good would you be interested in helping us out with our mod that is going to change everything for both lan and dedicated servers? Please MSG me if your interested. If you need to see the code dev first here you go:
https://github.com/jredfox/lanessentials

Edited July 4, 2018 by jredfox

[Draco18s]

8 hours ago, Jaideep Heer said:

If you use a ClassTransformer to add an instruction

No.

We should not be telling people to do this.

Especially people like jredfox.

[Cadiboo]

If people don’t know how something works... they should learn how it works before trying to change anything to do with it. Giving them software that tries to get around this will never solve the problem, especially if we’re dealing with stuff to deal with the internet, as everything on the internet is jury rigged