Sitelet https://github.com/transparency-dev/formats/pull/125
Skip to content

Drop dep on the certificate-transparency-go repo - #125

Merged
AlCutter merged 3 commits into
transparency-dev:mainfrom
AlCutter:no_ct_go
Jun 10, 2024
Merged

AlCutter merged 3 commits into
transparency-dev:mainfrom
AlCutter:no_ct_go

Conversation

@AlCutter

@AlCutter AlCutter commented May 24, 2024 •

Copy link
Copy Markdown
Collaborator

This PR removes the temporary dep on the certificate-transparency-go repo by implementing serialisation of STH signature preimages locally.

Fises #105

@AlCutter
AlCutter requested a review from a team as a code owner May 24, 2024 13:32
@AlCutter
AlCutter requested a review from jiggoha May 24, 2024 13:32
@codecov-commenter

codecov-commenter commented May 24, 2024 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.47%. Comparing base (fa00c16) to head (260df25).
⚠️ Report is 156 commits behind head on main.

Files with missing lines Patch % Lines
note/note_rfc6962.go 66.66% 3 Missing and 1 partial ⚠️

❗ There is a different number of reports uploaded between BASE (fa00c16) and HEAD (260df25). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (fa00c16) HEAD (260df25)
4 2
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
- Coverage   82.57%   76.47%   -6.11%     
==========================================
  Files           5        6       +1     
  Lines         241      323      +82     
==========================================
+ Hits          199      247      +48     
- Misses         30       45      +15     
- Partials       12       31      +19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AlCutter

Copy link
Copy Markdown
Collaborator Author

@mhutchinson need a re-approval as I had to rebase and resolve go.mod.

Comment thread note/note_rfc6962.go Outdated
"time"

ct "github.com/google/certificate-transparency-go"
//"github.com/google/certificate-transparency-go/tls"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this comment serving any value?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

gah, will need another re-approve.

@AlCutter
AlCutter merged commit 01e8727 into transparency-dev:main Jun 10, 2024
@AlCutter
AlCutter deleted the no_ct_go branch June 10, 2024 13:01
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