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

Root ca #261

Merged
merged 5 commits into from
May 9, 2022
Merged

Root ca #261

merged 5 commits into from
May 9, 2022

Conversation

HarshGandhi-AWS
Copy link
Contributor

Motivation

  • Please give a brief description for the background of this change.
  • We should not validate code while parsing. We have a separate method Validate for this purpose.
  • Device Client should fail only if invalid RootCA file path is passed. Since it is not an mandatory field anymore, we should should not fail if the user fails to pass value for it.

Modifications

Change summary

Please describe what changes are included in this pull request.

  • Removed redundant code and moved it to Validate method in config.cpp
  • Updated the CONFIG.md documentation file.

Revision diff summary

If there is more than one revision, please explain what has been changed since the last revision.

Testing

Is your change tested? If not, please justify the reason.
Please list your testing steps and test results.

  • Added enough unit tests for testing my changes.
  • CI test run result:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@shangabl
Copy link
Contributor

shangabl commented May 7, 2022

I made all these changes so that Greengrass Secure Tunneling component would work. I agree we should update the README but the rest of the changes are just undoing the changes that fixed Greengrass.

shangabl
shangabl previously approved these changes May 9, 2022
docs/CONFIG.md Outdated Show resolved Hide resolved
@HarshGandhi-AWS HarshGandhi-AWS merged commit 5f9d4ad into main May 9, 2022
@HarshGandhi-AWS HarshGandhi-AWS deleted the rootCA branch May 9, 2022 17:56
RogerZhongAWS pushed a commit that referenced this pull request May 9, 2022
* updated root-ca validation to fail only if invalid path is passed.

* updated the Readme file

* corrected indentation errors

* removed few changes since it will break GG ST binary

* corrected typo
RogerZhongAWS pushed a commit that referenced this pull request May 11, 2022
* updated root-ca validation to fail only if invalid path is passed.

* updated the Readme file

* corrected indentation errors

* removed few changes since it will break GG ST binary

* corrected typo
joenghoyin pushed a commit to eguanatech/aws-iot-device-client-deprecated that referenced this pull request Aug 29, 2023
* updated root-ca validation to fail only if invalid path is passed.

* updated the Readme file

* corrected indentation errors

* removed few changes since it will break GG ST binary

* corrected typo
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.

3 participants