-
-
Notifications
You must be signed in to change notification settings - Fork 682
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Plugin Update Checker #1121
base: dev
Are you sure you want to change the base?
Conversation
LavalinkServer/src/main/java/lavalink/server/bootstrap/PluginManager.kt
Outdated
Show resolved
Hide resolved
3c3b547
to
26bf4bf
Compare
import java.nio.channels.Channels | ||
import java.util.* | ||
import java.util.jar.JarFile | ||
import javax.xml.parsers.DocumentBuilderFactory |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wasn't this removed on java 11?
https://www.oracle.com/java/technologies/javase/11-relnote-issues.html#JDK-8190378
either jackson xml or kotlinx.serialization should do the trick here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no that looks like it only affects the java namespace, javax looks like its untouched by that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather have this use a modern lib like the above
Adds a plugin update checker function to
PluginManager.kt
that runs on application startupCloses #1044
The function checks the maven repository of each plugin declaration and searches the maven metadata file for the latest version. If then compares the latest version and current version installed and writes to the console if the user needs to update. The 3rd party library "semver" is used as a dependency to compare versions.