Missing RCON Password Validation #3

New Issue

2026-02-02T15:09:46-05:00

jared commented

2026-02-02 15:09:46 -05:00

if not MINECRAFT_RCON_PASSWORD: # Line 410
# Only checked after user interaction
Issue: Should validate on startup, not during command execution.

Fix:

In CustomBot.init or setup_hook

async def setup_hook(self):
# Validate critical configs
if not MINECRAFT_RCON_PASSWORD:
logger.warning("MINECRAFT_RCON_PASSWORD not set - /minecraft command will fail")
if not PELICAN_API_KEY:
logger.warning("PELICAN_API_KEY not set - Pelican features disabled")

await self.tree.sync()

if not MINECRAFT_RCON_PASSWORD: # Line 410 # Only checked after user interaction Issue: Should validate on startup, not during command execution. Fix: # In CustomBot.__init__ or setup_hook async def setup_hook(self): # Validate critical configs if not MINECRAFT_RCON_PASSWORD: logger.warning("MINECRAFT_RCON_PASSWORD not set - /minecraft command will fail") if not PELICAN_API_KEY: logger.warning("PELICAN_API_KEY not set - Pelican features disabled") await self.tree.sync()

jared referenced this issue from a commit

2026-02-10 20:42:30 -05:00

Refactor bot with 17 issue fixes: logging, caching, error handling, security

jared closed this issue

2026-02-10 20:42:30 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: LotusGuild/discordBot#3