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 CACert attribute to connection #203

Merged
merged 1 commit into from
Mar 20, 2020
Merged

Conversation

masayag
Copy link
Contributor

@masayag masayag commented Mar 19, 2020

Adding the option to provide CACert as a []byte in addition to file.
In certain cases, the client might not have the certificate created in a
file. Connection builder should support CA Cert without requiring a file
by accepting its value as a parameter.

Signed-off-by: Moti Asayag [email protected]

Adding the option to provide CACert as a []byte in addition to file.
In certain cases, the client might not have the certificate created in a
file. Connection builder should support CA Cert without requiring a file
by accepting its value as a parameter.

Signed-off-by: Moti Asayag <[email protected]>
@pkliczewski
Copy link
Contributor

@imjoey please take a look

@imjoey
Copy link
Member

imjoey commented Mar 20, 2020

@masayag this is a great feature. LGTM. Let us merge it. Thank you so much for your contribution.

@imjoey imjoey merged commit 4808055 into oVirt:master Mar 20, 2020
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