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

Add support for custom certificate stores #1420

Merged
merged 7 commits into from
Jan 17, 2022

Conversation

mheege-abb
Copy link
Contributor

Pull request for #1419

@CLAassistant
Copy link

CLAassistant commented Jun 7, 2021

CLA assistant check
All committers have signed the CLA.

@mregen
Copy link
Contributor

mregen commented Jun 11, 2021

Hi @mheege-abb , this is a really nice idea and we had this type of feature on the backlog for quite some time.
However, we need to ensure it works really well with the configuration tools and there are no issues loading e.g. private keys.
I'd suggest a memory based certificate store (we need this anyway for test tooling) as a sample that can be used to test this new interface in depth. Also certvalidator tests should be able to run with that sample cert store, so somebody can test his custom implementation. Would you be able to provide this sample store and tests in addition to this PR?

Thanks, Martin

Copy link
Contributor

@mregen mregen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mheege-abb, please provide samples and tests with this change --> see my comments.

@mheege-abb
Copy link
Contributor Author

I will try to fix the typo and the naming and to implement a memory based certificate store this week, but I cannot promise.

Marcus

@mregen
Copy link
Contributor

mregen commented Jun 14, 2021

@mheege-abb great, thanks!

@mregen
Copy link
Contributor

mregen commented Dec 17, 2021

Targeting after next release... it is quite a task, specifically with ECC support coming up.

@mregen mregen added this to the 1.4.369 milestone Dec 17, 2021
@mheege-abb
Copy link
Contributor Author

Targeting after next release... it is quite a task, specifically with ECC support coming up.

Thanks for the info. I hope I will have time to write the tests in the next weeks.
How is the support for custom cert stores related to ECC support?
What is the expected time frame for the release following the next one?

Thanks

Marcus Heege

@mregen mregen modified the milestones: 1.4.369, 1.4.368 Jan 5, 2022
@mregen
Copy link
Contributor

mregen commented Jan 5, 2022

we will try to squeeze this into the 368 release, due by end of Jan if all goes well..

@mregen
Copy link
Contributor

mregen commented Jan 15, 2022

Hi @mheege-abb, I resolved the merge conflict to get it building. I think this looks good, do you consider your contribution final?

@mregen mregen marked this pull request as ready for review January 15, 2022 18:04
@mregen
Copy link
Contributor

mregen commented Jan 16, 2022

hi @mheege-abb , while fixing the build I figured this is a good time to fix the ICertStore interface too, but to make it fully async introduces a massive number of changes, so I will put it in a different PR. Thanks a lot for your contribution!

@mheege-abb
Copy link
Contributor Author

Hi @mheege-abb, I resolved the merge conflict to get it building. I think this looks good, do you consider your contribution final?

Yes, please consider my contribution final.

Marcus Heege

@mregen mregen merged commit 61e204f into OPCFoundation:master Jan 17, 2022
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.

Request for enhancement: Add Support for custom certificate stores
4 participants