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

VCertTknClient read zone configuration issue 115 #116

Merged

Conversation

marcos-albornoz
Copy link
Contributor

@marcos-albornoz marcos-albornoz commented Sep 1, 2022

The issue #115 was happening due a mistake that caused the VCertTknClient SDK methods stop to work given in the background they started to use the declared endpoints for VCertClient.

It was also added a suite of tests to test the VCertTknClient SDK methods work correctly.

The VCertTknClient SDK methods left to work since the releases v.0.9.0
until v0.9.2 given a mistake which caused the endpoints used in the
background were the specific endpoints declared for the VCertClient.

Also it was created a bunch of tests to test the VCertTknClient SDK
methods in order to confirm they are working correctly.

Resolves: #115
Copy link

@luispresuelVenafi luispresuelVenafi left a comment

Choose a reason for hiding this comment

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

Changes looks good

@marcos-albornoz marcos-albornoz merged commit 8981df6 into master Sep 1, 2022
@tr1ck3r tr1ck3r deleted the VCertTknClient_readZoneConfiguration_issue_115 branch September 27, 2022 19:42
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.

4 participants