Мне кажется, такое уже не лечится.
public Component colorize(String message) {
if (message == null) return Component.empty();
return MiniMessage.miniMessage().deserialize(message);
}Раз уж зашло про Objects.requireNonNull при получении значений из конфига, то думаю что тогда стоит дополнить что для геттеров (getString, getInt, getBoolean и тд) можно указать дефолтное значение если параметра в конфиге нет, и тогда он НУ ВОТ УЖЕ ТОЧНО не выдаст null. А вообще ещё очень хочется добавить про наименование классов, пакетов, методов и тд
Какие уязвимости это образует?
никаких
Дело не только в equals/hashCode, но и в том, какие поля хранятся в этом объекте. Если плагин хранит объект игрока и после его выхода, может остаться то, что должен был унести с собой сборщик мусора. Поэтому, хоть хранить их и можно, лучше использовать weak reference (для этого есть, в том числе и специализированные коллекции)
Хм, вообще, в целом, частично то это затронуто, хотя e.isCancelled реально были на моей памяти... вероятно запишу
