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

Update crypto to v0.31.0 #10498

Closed
wants to merge 2 commits into from

Conversation

ashishb-solo
Copy link

Description

Update golang.org/x/crypto to address CVE

API changes

Code changes

CI changes

Docs changes

Context

Interesting decisions

Testing steps

Notes for reviewers

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

@solo-changelog-bot
Copy link

Issues linked to changelog:
#10488

@ashishb-solo
Copy link
Author

ashishb-solo commented Dec 18, 2024

/kick

trying to get rid of this error:

Step #1 - "run-tests": Step #3 - "prepare-aws-creds": ERROR: (gcloud.kms.decrypt) Failed to read ciphertext file [./ci/aws_credentials.enc]: Unable to read file [./ci/aws_credentials.enc]: [Errno 2] No such file or directory: './ci/aws_credentials.enc'

it doesn't seem to be affecting other pull requests, so hopefully it was a transient issue

@ashishb-solo
Copy link
Author

@nfuden mentioned that buildbot will not work because the branch name has to be gloo. ergo i'm going to close this pull request and open a new one off a different branch.

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.

1 participant