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

Added support for WPA2 Enterprise, including example code. #34

Merged
merged 4 commits into from
Apr 7, 2019

Conversation

docmollo
Copy link
Contributor

@docmollo docmollo commented Apr 5, 2019

This is the CircuiPython code that matches the NINA firmware PR I made yesterday. I've included an example script, as well. I've tested this on a PyPortal and Metro M4, both running CircuitPython 4.0.0-beta.6. I did my best to match the existing style, but I may have not have gotten it right. Let me know if you'd like me to tweak things.

@ladyada
Copy link
Member

ladyada commented Apr 5, 2019

structure looks good to me! once its linted we can merge :)

@docmollo
Copy link
Contributor Author

docmollo commented Apr 5, 2019

That was fast....and disapointing.. Looks like it didn't like the line endings?

@ladyada
Copy link
Member

ladyada commented Apr 5, 2019

try running dos2unix

@docmollo
Copy link
Contributor Author

docmollo commented Apr 5, 2019

One thought I just had... I didin't change the version number in the NINA firmware. I was going to mention something about that, but forgot. The version number in the firmware should probably be rev'd in order to make debugging easier. I could update the example script to contain a message about that new version requirement. And perhaps the README.md could/should contain that requirement.

@ladyada
Copy link
Member

ladyada commented Apr 6, 2019

yep please update the version number too!

… updated README.rst to recommended checking version dependencies in example code.
@ladyada ladyada merged commit 607697c into adafruit:master Apr 7, 2019
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Apr 25, 2019
Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP32SPI to 1.3.7 from 1.3.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP32SPI#39 from brentru/update-simpletest
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP32SPI#38 from cpforbes/add_gpio
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP32SPI#34 from docmollo/docs_wpa2_enterprise

Updating https://github.com/adafruit/Adafruit_CircuitPython_PyPortal to 2.1.1 from 2.1.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_PyPortal#27 from brentru/add-image-http-path
  > Merge pull request adafruit/Adafruit_CircuitPython_PyPortal#26 from DigitalMakerCIC/master
@docmollo docmollo deleted the docs_wpa2_enterprise branch May 11, 2019 21:25
@nezra nezra mentioned this pull request May 23, 2019
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

Successfully merging this pull request may close these issues.

None yet

2 participants