Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Fix - The Forest #2883

Closed
wants to merge 4 commits into from
Closed

Fix - The Forest #2883

wants to merge 4 commits into from

Conversation

ImKringle
Copy link
Contributor

Description

As of now the File parse type reads both the serverPassword and serverPasswordAdmin section of the config.cfg as just the main server password. Doing this causes a crash on startup as two sections of the config are the same thing

This edit changes it to not include the serverPasswordAdmin in the config and instead sets it in the Startup Command

This edit is for:

  1. Removes serverPasswordAdmin from config creation on install
  2. Adds -serverPasswordAdmin {{ADMIN_PASS}} to startup
  3. Removes serverPasswordAdmin from the configuration files section of the egg

Checklist for all submissions

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you tested and reviewed your changes with confidence that everything works?
  • Did you branch your changes and PR from that branch and not from your master branch?
    • If not, why?:
  • You verify that the start command applied does not use a shell script
    • If some script is needed then it is part of a current yolk or a PR to add one
  • The egg was exported from the panel

@ImKringle
Copy link
Contributor Author

erm.. okay this for some reason includes my PR for #2869 - Lemme fix that >.>

@ImKringle ImKringle closed this Apr 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant