The openport command has been temporarily removed. This fixes the nullpointer error and allows me to rewrite the admittedly poor implementation.
This version bumps some internal dependencies and features nothing new for end users.
This version brings the removal of uneeded logging frameworks, and switches to the built in plugin logger. The plugin also now uses Spigot Plugin Annotations to generate the plugin.yml.
The plugin also now uses the Spigot api instead of the Paper api to ensure comptatability.
Plugin no longer incorrectly requires legacy material support.
This version fixes an issue where user defined ports were only opened to TCP, not UDP.
Finished removal of old code.
Removed broken code
There were originally plans to use the api from mcsrvstat.us to check if port mapping was successful but I have decided to forego this functionality for now. It may or may not be a feature of a future release.
The plugin will now load and should work on older server versions. As a result you may now see the following in your logs:
[12:02:55 WARN]: [org.bukkit.craftbukkit.v1_19_R3.legacy.CraftLegacy] Initializing Legacy Material Support. Unless you have legacy plugins and/or data this is a bug!
[12:02:57 WARN]: Legacy plugin UPnP v1.4-SNAPSHOT does not specify an api-version.
This is a result of allowing the plugin to support multiple versions in one file.
Fixed a bug that prevented new ports from being mapped.
The plugin will now forward both TCP and UDP for the Minecraft server's port.
Also improved code readability.