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

AP password not settable #34

Closed
W-Geronius opened this issue Mar 10, 2022 · 2 comments
Closed

AP password not settable #34

W-Geronius opened this issue Mar 10, 2022 · 2 comments

Comments

@W-Geronius
Copy link

Whatever I try, I cannot change the AP passwort (I have been trying on OBP60).
Modifying Admin Pass + enabling/disabling seem to work.

Attached Screenshot and debug-log was produced after factory reset. Changes to config as shown (testpass, testadmin)
image

Debug log:
2022-03-10_111325 COM8 ESP32 wroom.zip

After restart, the network name was changed. Logging in however does not work with the new password. It had remained the default.
Subsequent changes have no effect either.

@wellenvogel
Copy link
Owner

Currently it works as designed (more or less).
As long as there is no hardware reset button (the define GWBUTTON_PIN must be set) the changing of the AP pass will not be allowed.
And the OBP60 does not set it currently.
But it should also not be visible in the UI in this case.
Additionally a flag FORCE_AP_PWCHANGE will be introduced to allow other software (like the OBP60) to force the ability of an AP password change (with the risk for the user to lock itself out and requiring a full reflash).

wellenvogel pushed a commit that referenced this issue Mar 10, 2022
…owed. Also do not print this on startup. Introduce FORCE_AP_PWCHANGE to allow extensions to force pw change ability
@W-Geronius
Copy link
Author

setting apPassword works fine in dev 20220311. Issue may be closed from my POV, thanks

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

No branches or pull requests

2 participants