Skip to content

Conversation

@BoredHF
Copy link
Contributor

@BoredHF BoredHF commented Dec 30, 2025

Broken system user IDs fixed in config. Change how (uid and gid) are handled in the configuration, ensuring they always have sensible default values and improving logging for better traceability. The changes introduce default values for user IDs, add utility functions to enforce these defaults, and log the final values used.

Configuration defaults and enforcement:

  • Added default values (8888) for uid and gid in the SystemConfiguration struct to ensure they are set if not provided in the configuration file.
  • Introduced a defaultUserID constant and utility function ensureDefaultUserIDs to automatically assign default values to uid and gid if they are zero at runtime. [1] [2]

Logging improvements:

  • Added a logFinalIDs function to log the final uid and gid values being used, improving transparency and debugging.

Application of defaults in user setup:

  • Updated the EnsurePterodactylUser function to call ensureDefaultUserIDs and logFinalIDs at key points, ensuring defaults are applied and logged regardless of the environment (Docker, normal, or Alpine Linux). [1] [2] [3] [4]

@naterfute naterfute merged commit 855bc1d into pyrohost:main Jan 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants