Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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.

Sign up for GitHub

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

Jump to bottom

Rainmaker WiFi provisioning: Add NVS counter of WiFi connect failures #50

Closed
martinberlin opened this issue Dec 9, 2021 · 1 comment
Closed

Rainmaker WiFi provisioning: Add NVS counter of WiFi connect failures #50

martinberlin opened this issue Dec 9, 2021 · 1 comment
Assignees
@martinberlin
Labels
doing Actually working on this needs Testing Not yet tested, cannot be used in prooduction

Comments

@martinberlin
Copy link
Owner

Is your feature request related to a problem? Please describe.
When using Rainmaker, if the user changes the router configuration credentials, without clearing credentials first in ESP rainmaker app then it enters in a restart loop. Because it cannot connect but also can’t clear the WiFi configuration in order to entern Blietppth provisioning mode.

Describe the solution you'd like
The solution would be to have a WiFi failed connect counter that is stored in Non Volatine Storage, that when reaches 100 failed connections, clears the WiFi configuration and restarts. The issue with this is, is that it might clear credentials in a moment where the user does not want, simply becau It’s normal that from time to time the connect to WiFi process fails.
@martinberlin martinberlin added the enhancement New feature or request label Dec 9, 2021
@martinberlin martinberlin self-assigned this Dec 9, 2021
@martinberlin martinberlin changed the title Add NVS counter of WiFi connect failures Rainmaker WiFi provisioning: Add NVS counter of WiFi connect failures Dec 9, 2021
@martinberlin martinberlin pinned this issue Dec 13, 2021
@martinberlin martinberlin mentioned this issue Dec 14, 2021
Closed
3 tasks
@martinberlin martinberlin added on hold Will be probably addressed later and removed enhancement New feature or request labels Sep 1, 2022
@martinberlin
Copy link
Owner Author

On hold till we upgrade this to IDF v5.0 and revise how Rainmaker works on that version

@martinberlin martinberlin unpinned this issue Sep 1, 2022
@martinberlin martinberlin added needs Testing Not yet tested, cannot be used in prooduction doing Actually working on this and removed on hold Will be probably addressed later labels Oct 17, 2022
martinberlin added a commit that referenced this issue Oct 17, 2022
@martinberlin
#50 Rainmaker is now working on ESP-IDF v5
17ed05c
Repository owner locked and limited conversation to collaborators Dec 2, 2022
@martinberlin martinberlin converted this issue into discussion #77 Dec 2, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Assignees

@martinberlin martinberlin

Labels
doing Actually working on this needs Testing Not yet tested, cannot be used in prooduction
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@martinberlin