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

Include source repo visibility in oid range #1302

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

jonjohnsonjr
Copy link
Contributor

This appears to be a DER-encoded string with the ASN.1 UTF8String tag.

Fixes #1301

Summary

This seems like an oversight or typo.

Release Note

NONE

Documentation

NONE

This appears to be a DER-encoded string with the ASN.1 UTF8String tag.

Signed-off-by: Jon Johnson <[email protected]>
@haydentherapper haydentherapper enabled auto-merge (squash) August 1, 2023 23:15
@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Merging #1302 (f1e895e) into main (1bf2028) will decrease coverage by 0.17%.
Report is 3 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1302      +/-   ##
==========================================
- Coverage   58.49%   58.33%   -0.17%     
==========================================
  Files          50       50              
  Lines        3053     3053              
==========================================
- Hits         1786     1781       -5     
- Misses       1110     1114       +4     
- Partials      157      158       +1     

see 1 file with indirect coverage changes

@haydentherapper haydentherapper merged commit 82c9843 into sigstore:main Aug 1, 2023
13 checks passed
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.

There is a typo
2 participants