MCCTeam / Minecraft-Console-Client

Lightweight console for Minecraft chat and automated scripts
https://mccteam.github.io
Other
1.67k stars 402 forks source link

[BUG] MCC won't run if Sentry is disabled #2779

Closed ObscenityIB closed 3 months ago

ObscenityIB commented 3 months ago

Prerequisites

Minecraft Version

N/A

Console Client Version

ca966a464c2262a7b51f6395fe901b12b832e7ee

Expected Behavior

MCC to run with analytics disabled.

Actual Behavior

MCC crashes if analytics are not enabled.

Steps to Reproduce the bug

  1. Edit default ini to set EnableSentry=false
  2. Run MCC
  3. Crash

Attach screenshot here (If applicable)

No response

Anythings that could help diagnosing the bug

Minecraft Console Client v1.20.4 - for MC 1.4.6 to 1.20.4 - Github.com/MCCTeam
[MCC] [ScriptScheduler] Loaded task:
triggeronfirstlogin: False
 triggeronlogin: False
 triggerontime: True
 triggeroninterval: True
 timevalue: 14:00:00
 timeinterval: 53.4
 action: send /hello
[MCC] [ScriptScheduler] Loaded task:
triggeronfirstlogin: False
 triggeronlogin: True
 triggerontime: False
 triggeroninterval: False
 timevalue: 
 timeinterval: 56
 action: send /login pass
Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
   at MinecraftClient.Program.Main(String[] args)
Aborted (core dumped)


### Device

Laptop

### Operating System

Linux

### Server Address (If applicable)

_No response_
breadbyte commented 3 months ago

Could not reproduce with the build from https://github.com/MCCTeam/Minecraft-Console-Client/releases/tag/20240713-273, on both Windows and Linux.

Did you build MCC from source?

ObscenityIB commented 3 months ago

Yes, compiled from ca966a464c2262a7b51f6395fe901b12b832e7ee using dotnet publish MinecraftClient -f net7.0 -r linux-x64 --no-self-contained -c Release -p:UseAppHost=true -p:IncludeNativeLibrariesForSelfExtract=true -p:DebugType=None.

log-diag.txt

breadbyte commented 3 months ago

Should be fixed in c69cdad. Thanks for the report!