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

Fix type inference regression in indirect dependency #156

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

omertuc
Copy link
Member

@omertuc omertuc commented Aug 1, 2024

Background

Recert uses the simple_asn1 crate in order to calculate the SKID on RSA keys

Problem

The simple_asn1 crate depends on the time crate, which recently encountered a regression [1] due to [2] that caused the simple_asn1 crate to fail to compile with our locked version (0.3.30) of the time crate.

Solution

Updated the time crate to version 0.3.36, which contains a fix for the regression. Re-ran vendoring.

[1] time-rs/time#681
[2] rust-lang/rust#125319

# Background

Recert uses the `simple_asn1` crate in order to calculate the SKID on
RSA keys

# Problem

The `simple_asn1` crate depends on the `time` crate, which recently
encountered a regression [1] due to [2] that caused the `simple_asn1`
crate to fail to compile with our locked version (0.3.30) of the `time`
crate.

# Solution

Updated the `time` crate to version 0.3.36, which contains a fix for the
regression. Re-ran vendoring.

[1] time-rs/time#681
[2] rust-lang/rust#125319
@openshift-ci openshift-ci bot requested review from mresvanis and tsorya August 1, 2024 08:29
Copy link

openshift-ci bot commented Aug 1, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: omertuc

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label Aug 1, 2024
@eranco74
Copy link
Collaborator

eranco74 commented Aug 1, 2024

/lgtm

@eranco74
Copy link
Collaborator

eranco74 commented Aug 1, 2024

/retest-required

@openshift-merge-bot openshift-merge-bot bot merged commit 027c3e1 into rh-ecosystem-edge:main Aug 1, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants