jmurth1234 / JPanel

A plugin based control panel for Bukkit / Spigot
http://dev.bukkit.org/bukkit-plugins/jpanel/
Do What The F*ck You Want To Public License
26 stars 9 forks source link

Setting SSL to true causes the server to shutdown #9

Open RoboMWM opened 8 years ago

RoboMWM commented 8 years ago

Yea I know I don't have a SSL certificate, but something odd is causing the server to instantly shut down the moment the web socket is enabled:

[14:27:24] [Server thread/INFO]: [JPanel] Enabling JPanel v1.0-SNAPSHOT
[14:27:28] [Server thread/WARN]: SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
[14:27:28] [Server thread/WARN]: SLF4J: Defaulting to no-operation (NOP) logger implementation
[14:27:28] [Server thread/WARN]: SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
[14:27:28] [Server thread/INFO]: [JPanel] JPanel enabled!
[14:27:28] [Server thread/INFO]: [BKCommonLib] Enabling BKCommonLib v1.70-SNAPSHOT
[14:27:28] [Server thread/INFO]: [BKCommonLib] BKCommonLib is running on Spigot (git-Paper-854) : v1_10_R1 (Minecraft 1.10.2)
[14:27:28] [Server thread/INFO]: [BKCommonLib] [Network] Now using the ProtocolLib library to provide Packet Listener and Monitor support
[14:27:28] [Server thread/INFO]: [BKCommonLib] Not enough wrappers. Needs more wrappers. Moooreee...
[14:27:28] [Server thread/INFO]: BKCommonLib version 1.70-SNAPSHOT enabled! (0.099s)
[14:27:28] [Server thread/INFO]: [mcMMO] Enabling mcMMO v1.5.07-SNAPSHOT-b${BUILD_NUMBER}
[14:27:28] [Thread-9/INFO]: [JPanel] WebSocket started
[14:27:28] [Thread-4/INFO]: Stopping server
MuhsinunC commented 7 years ago

Here's my issue with the starting sequence; dunno if it's related or not: [17:18:19] [Server thread/INFO]: [JPanel] Enabling JPanel v1.0-SNAPSHOT [17:18:19] [Server thread/WARN]: log4j:WARN No appenders could be found for logger (spark.staticfiles.StaticFilesConfiguration). [17:18:19] [Server thread/WARN]: log4j:WARN Please initialize the log4j system properly. [17:18:19] [Server thread/INFO]: [JPanel] JPanel enabled!

RoboMWM commented 7 years ago

@MuhsinunCool no it isn't, unless your issue also involves enabling SSL, and the server stopping the moment the websocket is started.

MuhsinunC commented 7 years ago

@RoboMWM Just thot that since urs sed this: [14:27:28] [Server thread/WARN]: SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder". [14:27:28] [Server thread/WARN]: SLF4J: Defaulting to no-operation (NOP) logger implementation [14:27:28] [Server thread/WARN]: SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details. and so did mine, I just thot that they might've been related idk :)

RoboMWM commented 7 years ago

Not related, I get that regardless of the SSL setting I use.

jmurth1234 commented 7 years ago

Have you followed the instructions for using https?

From the bukkitdev page:

HTTPS support

In the latest beta (Beta 6) I've added standalone https support! This requires some configuration.

  1. First, you'll need to put a keystore file in the JPanel folder. You can either attempt to generate one using these instructions or download a premade keystore file from here. The pre-made keystore has an unsigned certificate that'll get you going with security warnings in most browsers, so isn't recommended unless you can't use an actual certificate.

  2. Stop the server, and enter the name of the keystore file (with file extension) under keystore-name and the password for the keystore file under keystore-password in the JPanel config.yml. If you're using the premade keystore, use the password 'password' (ultra secure).

  3. Restart the server, then access the panel normally except with https.

RoboMWM commented 6 years ago

JPanel is also causing the server to stop when it's reloaded. Seeing that the message comes from another thread (not the main thread), my guess is that jetty or whatever is calling a System#exit or similar?

ghost commented 6 years ago

It's not because of setting ssl to true. It's hard coded into the plugin. Use ploader | bypass plugin limit restrictions to work around that issue.

On Tue, Aug 28, 2018, 4:14 PM RoboMWM notifications@github.com wrote:

JPanel is also causing the server to stop when it's reloaded. Seeing that the message comes from another thread (not the main thread), my guess is that jetty or whatever is calling a System#exit or similar?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/rymate1234/JPanel/issues/9#issuecomment-416742945, or mute the thread https://github.com/notifications/unsubscribe-auth/AL_oO7XU3mBRoRxoQg5FTUPKPO2-yttHks5uVbLNgaJpZM4J3YPP .

RoboMWM commented 6 years ago

No idea what you're talking about here. What plugin limit restriction are you talking about? Where is this hardcoded into the plugin? (I did searches for "shutdown" with no results in github search).

RoboMWM commented 6 years ago

Also, not saying it's the root cause, but setting SSL to true without having a cert setup does cause the plugin (or whatever library) to shutdown the server.

ghost commented 6 years ago

I guess @rymate1234 removed the Bukkit#shutdown call in onDisable in a recent commit, but once upon a time it was there and that's what I was referring to. ploader-bypass-plugin-limit-restrictions.56669/

RoboMWM commented 6 years ago

He does have stop() (or did, idk rn), but I never saw shutdown(). Would you know what version had it? Because there was no shutdown() existing in the codebase last year

RoboMWM commented 6 years ago

No idea what that plugin is for other than allowing you to blacklist plugins that should not be reloaded when running a server reload. I do not use /reload but an arguably even eviler /plugman reload :p

RoboMWM commented 6 years ago

Also I should add that this has nothing to do with reloading the plugin, nor is the shutdown method called from the main thread given the provided log.