Added Nexus CM Procedures as Product ID's; conditional sync functionality; cleanup; unit tests#6
Merged
Merged
Conversation
… (error when there is no default set).
flattening exception to retain potential useful info Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…onal sync functionality; cleanup; unit tests
Updated workflow to use version 3 of the starter workflow and modified secrets.
…r/nexus-certificate-manager-caplugin into add_procedures_AB#81415
indrora
added a commit
that referenced
this pull request
May 21, 2026
…lity; cleanup; unit tests (#6) (#7) * Implemented enrollment, added helper methods and API response handlers * implemented cert retrieval methods on client. * added additional logging, implemented revoke on client * implemented sync and revoke; completed initial functionality * added changelog and license headers * added manifest, disabled auth cert domain check for nexus auth cert * Updated enrollment to include first available procname for enrollment (error when there is no default set). * updated request format for revocation * cleanup * documentation updates * updated project settings for github build * added keyfactor-bootstrap-workflow.yml * updated manifest * added docsource folder * corrected the returned value on a revoke request * Update nexus-certificate-manager-caplugin/NexusCertManagerCAPlugin.cs flattening exception to retain potential useful info * Update docsource/configuration.md * Update nexus-certificate-manager-caplugin/NexusCertManagerClient.cs * Update nexus-certificate-manager-caplugin/models/Helpers.cs * Update nexus-certificate-manager-caplugin/NexusCertManagerCAPlugin.cs * Update nexus-certificate-manager-caplugin/NexusCertManagerClient.cs * Update nexus-certificate-manager-caplugin/NexusCertManagerCAPlugin.cs * Update nexus-certificate-manager-caplugin/NexusCertManagerClient.cs * Update nexus-certificate-manager-caplugin/NexusCertManagerCAPlugin.cs * added check for partial sync * Update nexus-certificate-manager-caplugin/NexusCertManagerCAPlugin.cs * updating manifest for doctool build * added configuration.md to solution * updated gitignore * Now returning Nexus CM processes as product ID's; implemented conditional sync functionality; cleanup; unit tests * Update generated docs * Change starter workflow version and update secrets Updated workflow to use version 3 of the starter workflow and modified secrets. * added .net6.0 build * removed leftover comment from configuration.md * Update generated docs --------- Co-authored-by: Joe VanWanzeele <76071503+joevanwanzeeleKF@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Keyfactor <keyfactor@keyfactor.github.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
GetProductIds()now returns Nexus CA token procedure names dynamically from the/proceduresendpoint rather than a single hardcoded value. Each certificate template in Command should be configured with a procedure name as its ProductID.SynchronizeandGetCertificateListnow page through results in batches of 500, resolving failures that occurred when the max returned records limit (that defaults to 500) was reached.SynchronizethrowsNotSupportedExceptionwith a clear explanation whenSyncProcedureFieldis not configured. When configured, sync reads the specifiedExtendedCertSearchfield from each certificate to resolve its ProductID. See documentation for CA-side requirements.GetSingleRecordfix —ProductIDis now resolved from the configuredExtendedCertSearchfield instead of incorrectly usingCertId.ValidateProductInfo— now validates thatProductIDis non-empty..Resultwith.GetAwaiter().GetResult()inGetProductIds().