Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Admin panel defaults Maintenance Mode setting to value of Behind Proxy #4926

Open
3 tasks done
danny6167 opened this issue Nov 18, 2023 · 0 comments · May be fixed by #4927
Open
3 tasks done

Admin panel defaults Maintenance Mode setting to value of Behind Proxy #4926

danny6167 opened this issue Nov 18, 2023 · 0 comments · May be fixed by #4927
Labels
bug Something that's not working as it's intended to be.

Comments

@danny6167
Copy link
Member

Current Behavior

In the nodes settings page the current value of Maintenance Mode is set to the value of Behind Proxy when the form is submitted.
This is due to the wrong key being passed to old() in the template

Expected Behavior

Save nodes settings page and the Maintenance Mode toggle value remains set to the value it had when the form was submitted

Steps to Reproduce

Open nodes settings page.
turn on behind proxy mode and save changes.
Observe that the Maintenance Mode value has also changed to match

Panel Version

1.11.5

Wings Version

N/A

Games and/or Eggs Affected

N/A

Docker Image

N/A

Error Logs

N/A

Is there an existing issue for this?

  • I have searched the existing issues before opening this issue.
  • I have provided all relevant details, including the specific game and Docker images I am using if this issue is related to running a server.
  • I have checked in the Discord server and believe this is a bug with the software, and not a configuration issue with my specific system.
@danny6167 danny6167 added the not confirmed Report seems plausible but requires additional testing or 3rd part confirmation. label Nov 18, 2023
@matthewpi matthewpi added bug Something that's not working as it's intended to be. and removed not confirmed Report seems plausible but requires additional testing or 3rd part confirmation. labels Nov 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that's not working as it's intended to be.
Projects
Status: 👀 In review
2 participants