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

Enterprise security support #11104

Conversation

@tejanarahari
Copy link

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 ARMmbed/mbed-os-maintainers Jul 24, 2019
@ciarmcom

This comment has been minimized.

Copy link
Member

commented Jul 24, 2019

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

Teja Narahari
@tejanarahari tejanarahari marked this pull request as ready for review Aug 6, 2019
@tejanarahari tejanarahari force-pushed the tejanarahari:pr/enterprise-security-feature-support-pr2 branch from df60f31 to d84b477 Aug 6, 2019
@tejanarahari

This comment has been minimized.

Copy link
Author

commented Aug 6, 2019

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

@maclobdell maclobdell requested review from ARMmbed/mbed-os-connectivity and removed request for ARMmbed/mbed-os-connectivity Aug 14, 2019
@maclobdell

This comment has been minimized.

Copy link
Contributor

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

This comment has been minimized.

Copy link

commented Aug 15, 2019

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

@tejanarahari

This comment has been minimized.

Copy link
Author

commented Aug 20, 2019

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

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

CI started

@mbed-ci

This comment has been minimized.

Copy link

commented Aug 21, 2019

Test run: SUCCESS

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

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Aug 21, 2019
@0xc0170 0xc0170 merged commit 476ce09 into ARMmbed:master Aug 21, 2019
27 checks passed
27 checks passed
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/jenkins/pr-merge Manually fixed status
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci/build-ARM Success
Details
jenkins-ci/build-GCC_ARM Success
Details
jenkins-ci/build-IAR Success
Details
jenkins-ci/cloud-client-test Success
Details
jenkins-ci/dynamic-memory-usage RTOS ROM(+0 bytes) RAM(+0 bytes)
Details
jenkins-ci/exporter Success
Details
jenkins-ci/greentea-test Success
Details
jenkins-ci/mbed2-build-ARM Success
Details
jenkins-ci/mbed2-build-GCC_ARM Success
Details
jenkins-ci/mbed2-build-IAR Success
Details
jenkins-ci/unittests Success
Details
travis-ci/astyle Success!
Details
travis-ci/docs Success!
Details
travis-ci/doxy-spellcheck Success!
Details
travis-ci/events Success! Runtime is 8568 cycles.
Details
travis-ci/gitattributestest Success!
Details
travis-ci/include_check Success!
Details
travis-ci/licence_check Success!
Details
travis-ci/littlefs Success! Code size is 8448B.
Details
travis-ci/psa-autogen Success!
Details
travis-ci/tools-py2.7 Success!
Details
travis-ci/tools-py3.5 Success!
Details
travis-ci/tools-py3.6 Success!
Details
travis-ci/tools-py3.7 Success!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants
You can’t perform that action at this time.