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.

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

Add security in the wifi.connect() #607

Closed
communix opened this issue Aug 28, 2023 · 3 comments
Closed

Add security in the wifi.connect() #607

communix opened this issue Aug 28, 2023 · 3 comments
Assignees
@communix
Milestone
1.1.0-beta.4

Comments

@communix
Copy link
Collaborator

Add security in the wifi.connect() function as optional.
Default is OPEN. And implement storage feature to save it in the storage
@communix communix added this to the 1.1.0-beta.4 milestone Aug 28, 2023
@communix communix self-assigned this Aug 28, 2023
@albertd
Copy link
Contributor

albertd commented Aug 30, 2023

The security flag was added in pull request #606, just add "security" in your connect object. Valid options are: WPA2_WPA_PSK, WPA2_PSK, WPA_PSK, WEP_PSK, OPEN

@communix
Copy link
Collaborator Author

communix commented Sep 1, 2023

@albertd Yes. I want to make it as optional. So I created it. BTW thank you so much for your contribution.

@albertd
Copy link
Contributor

albertd commented Sep 1, 2023

You're very welcome, more to come. Working on stability and also adding more features.

communix added a commit that referenced this issue Sep 3, 2023
@communix
Adding security in the wifi connectionInfo
9045148 
 - #607
 - Set defalt value to make it optional
 - Use storage if security is not set
 - Fix strcmp bug for wifi security
@communix communix mentioned this issue Sep 3, 2023
Merged
@communix communix closed this as completed Sep 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@communix communix

Labels
None yet
Projects
None yet
Milestone
1.1.0-beta.4
Development

No branches or pull requests
2 participants
@albertd @communix