Merge pull request #27 from minecraft8997/main

Some fixes

Author: minecraft8997

.github/workflows/d.yml

@@ -32,7 +32,7 @@ jobs:
         dub build
         
     - name: Upload
-      uses: actions/upload-artifact@v3
+      uses: actions/upload-artifact@v4
       with:
         name: build-${{matrix.os}}
         path: ${{github.workspace}}/mcyeti*

source/client.d

@@ -208,6 +208,20 @@ class Client : Player {
 		outBuffer ~= identification.CreateData();
 	}
 
+	private bool ValidateUsername(string username) {
+		if (username.length < 2 || username.length > 16) return false;
+
+		for (int i = 0; i < username.length; i++) {
+			char current = username[i];
+			if (current == '_' || current == '.') continue;
+			if (current >= '0' && current <= '9') continue;
+			if ((current >= 'a' && current <= 'z') || (current >= 'A' && current <= 'Z')) continue;
+			return false;
+		}
+
+		return true;
+	}
+
 	void Update(Server server) {
 		auto time = Clock.currTime().toUnixTime();
 
@@ -234,6 +248,11 @@ class Client : Player {
 					packet.FromData(inBuffer);
 					inBuffer = inBuffer[packet.GetSize() .. $];
 
+					if (!ValidateUsername(packet.username)) {
+						server.Kick(this, "Bad username");
+						return;
+					}
+
 					auto correctMppass = md5Of(
 						server.salt ~ packet.username
 					).BytesToString();
@@ -306,7 +325,6 @@ class Client : Player {
 
 					if (muted) {
 						SendMessage("&eYou are muted");
-						return;
 					}
 
 					server.SendGlobalMessage(