Skip to content

Enterprise security support #11104

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

Conversation

tejanarahari
Copy link

@tejanarahari tejanarahari commented Jul 24, 2019

Description

This PR provides the capability to Cypress PSoC 6 targets to connect to 802.1x WiFi networks.
Added NSAPI_SECURITY_WPA2_ENT type to nsapi_security enum.

  • Phrase helps to connect to an 802.1X capable network.
  • Phrase conforms to WPA2-AES and WPA-TKIP with enterprise security.

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[X] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Please suggest

Release Notes

Add NSAPI_SECURITY_WPA2_ENT type to nsapi_security enum.

  • Phrase helps to connect to an 802.1X capable network.
  • Phrase conforms to WPA2-AES and WPA-TKIP with enterprise security.

@ciarmcom ciarmcom requested review from a team July 24, 2019 15:00
@ciarmcom
Copy link
Member

@cy-mtna, thank you for your changes.
@ARMmbed/mbed-os-ipcore @ARMmbed/mbed-os-maintainers please review.

@tejanarahari tejanarahari marked this pull request as ready for review August 6, 2019 10:06
@tejanarahari tejanarahari force-pushed the pr/enterprise-security-feature-support-pr2 branch from df60f31 to d84b477 Compare August 6, 2019 10:53
@tejanarahari
Copy link
Author

@SeppoTakalo Please can you merge the patch as I have tested the feature on CYPRESS platforms

@maclobdell maclobdell requested review from a team and removed request for a team August 14, 2019 18:45
@maclobdell
Copy link
Contributor

maclobdell commented Aug 15, 2019

@ARMmbed/mbed-os-maintainers - I checked out this PR and manually compiled the tests. Everything built successfully (except a few tests with long file names failed with gcc). Can the CI be triggered again to keep this PR moving? Thanks!

@40Grit
Copy link

40Grit commented Aug 15, 2019

Thoughts on tests for this? Or should they be target specific?

@tejanarahari
Copy link
Author

Thoughts on tests for this? Or should they be target specific?

Changes are not target specific, but the funtionality is applicable and tested only on Cypress platforms.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 21, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Aug 21, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

@0xc0170 0xc0170 merged commit 476ce09 into ARMmbed:master Aug 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants