SteamWar/BungeeCore
Archiviert
13
2

Fabric Mod Sender Integration. Mod itself is not final #307

Zusammengeführt
Lixfel hat 12 Commits von fabric_mod_sender nach master 2022-03-26 10:01:51 +01:00 zusammengeführt
Nur Änderungen aus Commit 1675b1dc9c werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -67,12 +67,12 @@ public class Fabric extends BasicListener {
byte[] data = e.getData();
Utils.VarInt varInt = Utils.readVarInt(data,0);
data = Arrays.copyOfRange(data,varInt.length, data.length);
if(data.length != varInt.length + varInt.value) {
Lixfel markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Du könntest das Packet prinzipiell vereinfachen, indem du den String ohne Länge vorher schickst. Der String wäre dann das gesamte Packet. Ansonsten würde ich mir wünschen, dass du überprüfst, dass der VarInt tatsächlich die Länge des restlichen Packets enthält (und falls Fehler Bann), derzeit nutzt du ja den VarInt gar nicht.

Du könntest das Packet prinzipiell vereinfachen, indem du den String ohne Länge vorher schickst. Der String wäre dann das gesamte Packet. Ansonsten würde ich mir wünschen, dass du überprüfst, dass der VarInt tatsächlich die Länge des restlichen Packets enthält (und falls Fehler Bann), derzeit nutzt du ja den VarInt gar nicht.
Veraltet
Review

1 teres kann ich nicht tun, da dieser varint von minecraft direkt angepappt wird. zweites kann ich machen.

1 teres kann ich nicht tun, da dieser varint von minecraft direkt angepappt wird. zweites kann ich machen.
banPlayer(user,player);
}
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Hier kann eine CharsetException oder ähnliches auftreten, in dem Fall wurde (offensichtlich) auch am fabricmodsender herumgepfuscht.

Hier kann eine CharsetException oder ähnliches auftreten, in dem Fall wurde (offensichtlich) auch am fabricmodsender herumgepfuscht.
data = Arrays.copyOfRange(data,varInt.length, data.length);
String dataString = "";
try{
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

Kannst du hier nicht for(X modIdentifier : array) machen?

Kannst du hier nicht for(X modIdentifier : array) machen?