zhangyuheng/BlueMap
1
0
Fork 0
forked from mirror/BlueMap
Code Pull Requests Activity

Made webserver binding logging less confusing (#474)

Browse Source 
People often think "0.0.0.0" is the IP they should connect to when BlueMap logs `WebServer bound to: /0.0.0.0:8100`
Or they see the IPv6 address being logged, and think BlueMap is only running on IPv6.
This commit is contained in:
TechnicJelle 2023-09-07 08:41:50 +02:00 committed by GitHub
parent c407ba6bd5
commit 7047df73d0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
BlueMapCommon/src/main/java/de/bluecolored/bluemap/common/web/http/Server.java
View File

@ -28,10 +28,12 @@
import java.io.Closeable;
import java.io.IOException;
import java.net.InetSocketAddress;
import java.net.SocketAddress;
import java.nio.channels.*;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Objects;
public abstract class Server extends Thread implements Closeable, Runnable {
@ -52,7 +54,20 @@ public void bind(SocketAddress address) throws IOException {
server.bind(address);
this.server.add(server);
Logger.global.logInfo("WebServer bound to: " + server.getLocalAddress());
if (checkIfBoundToAllInterfaces(address)) {
Logger.global.logInfo("WebServer bound to all network interfaces on port " + ((InetSocketAddress) address).getPort());
} else {
Logger.global.logInfo("WebServer bound to: " + server.getLocalAddress());
}
}
private boolean checkIfBoundToAllInterfaces(SocketAddress address) {
if (address instanceof InetSocketAddress) {
InetSocketAddress inetAddress = (InetSocketAddress) address;
return Objects.equals(inetAddress.getAddress(), new InetSocketAddress(0).getAddress());
}
return false;
}
@Override