Skip to content

Conversation

@bdehamer
Copy link
Collaborator

@bdehamer bdehamer commented Dec 9, 2025

Updates the Rekor client class to accept a new majorApiVersion parameter that can be used to signal that TLog entries should be created against V2 of the Rekor API.

@bdehamer bdehamer requested a review from a team as a code owner December 9, 2025 22:18
@changeset-bot
Copy link

changeset-bot bot commented Dec 9, 2025

🦋 Changeset detected

Latest commit: 2165f40

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@bdehamer bdehamer force-pushed the bdehamer/rekor-witness-v2 branch from 4d9f806 to bb127a3 Compare December 9, 2025 22:19
@bdehamer bdehamer force-pushed the bdehamer/rekor-witness-v2 branch from bb127a3 to 39435da Compare December 9, 2025 22:53
this.tlogV1 = new TLogClient({
...options,
rekorBaseURL:
options.rekorBaseURL || /* istanbul ignore next */ DEFAULT_REKOR_URL,
Copy link
Contributor

@ejahnGithub ejahnGithub Dec 10, 2025

Choose a reason for hiding this comment

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

I believe we would have a following up PR for handling the rekorBaseURL and the majorApiVersion?

ejahnGithub
ejahnGithub previously approved these changes Dec 10, 2025
Copy link
Contributor

@ejahnGithub ejahnGithub left a comment

Choose a reason for hiding this comment

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

lgtm!

Signed-off-by: Brian DeHamer <[email protected]>
@bdehamer bdehamer merged commit d5f9333 into main Dec 10, 2025
23 of 25 checks passed
@bdehamer bdehamer deleted the bdehamer/rekor-witness-v2 branch December 10, 2025 17:14
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