hoylabs/OpenDTU-OnBattery

Some settings pages do not redirect to login

Closed this issue · 1 comments

What happened?

Noticed by chance that I could access the battery settings while I was logged out. The AC charger settings can also be accessed. One cannot save new settings, but these pages should, as all other do, redirect to the login screen instead of exposing the current battery/AC charger settings when not logged in.

To Reproduce Bug

Log out, navigate to Battery settings or AC charger settings in web UI.

Expected Behavior

When not logged in, redirect to login page when trying to access settings pages, in order to not expose current settings.

Install Method

Pre-Compiled binary from GitHub

What git-hash/version of OpenDTU?

48bfa7c

Relevant log/trace output

No response

The problem does not occur when deselecting "allow view-only" web interface. This probably means that the wrong function is used to check authentication.

No response

Please confirm the following

  • I believe this issue is a bug that affects all users of OpenDTU, not something specific to my installation.
  • I have already searched for relevant existing issues and discussions before opening this report.
  • I have updated the title field above with a concise description.

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.