samolego / SimpleAuth

Simple authentication mod for Fabric or Forge servers.
https://modrinth.com/mod/SimpleAuth
MIT License
39 stars 14 forks source link

[BUG] [Fabric] Server crash on command usage #19

Closed valeth closed 4 years ago

valeth commented 4 years ago

Mod Loader

Versions

Describe the bug Complete server crash if any user uses any command.

error log

To Reproduce Steps to reproduce the behavior:

  1. Launch minimal Minecraft server with fabric, fabric-api and this mod
  2. Log into server with user or RCON
  3. Type any command eg.: /register 12345 12345
  4. Observe server crash

Expected behavior Server does not crash and either shows an error message to the user, or registers them.

Additional context The test server was run in a docker container. This was tested in both online, and offline mode. The test user was included in ops.json with permission level 4.

samolego commented 4 years ago

Hi, thanks for reporting. /register must be run by a player, however, crash isn't expected if ran by RCON/console etc. I'll look into it.

As far as the player goes, it seems strange to me. Will look at that as well.

samolego commented 4 years ago

Seeing the crash report, it has to do with password hashing. Please refer to wiki and follow the instructions there.

I still can't tell whether it happened due to running in docker or system missing libc6.