Skip to content
This repository was archived by the owner on Nov 1, 2023. It is now read-only.

Conversation

@nharper285
Copy link
Contributor

@nharper285 nharper285 commented Feb 7, 2023

Summary of the Pull Request

What is this about?
OpenSSL version in-use had issues, upgrading.

https://rustsec.org/advisories/RUSTSEC-2023-0006
https://rustsec.org/advisories/RUSTSEC-2023-0007
https://rustsec.org/advisories/RUSTSEC-2023-0009
https://rustsec.org/advisories/RUSTSEC-2023-0010

PR Checklist

  • Applies to work item: #xxx
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

@codecov-commenter
Copy link

Codecov Report

Merging #2812 (480b00e) into main (c5700b4) will decrease coverage by 3.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2812      +/-   ##
==========================================
- Coverage   29.93%   26.80%   -3.13%     
==========================================
  Files         324      125     -199     
  Lines       38990    13236   -25754     
==========================================
- Hits        11671     3548    -8123     
+ Misses      27319     9688   -17631     
Impacted Files Coverage Δ
src/agent/onefuzz/src/asan.rs
src/agent/libclusterfuzz/src/lib.rs
src/agent/coverage-legacy/src/code/tests.rs
src/agent/onefuzz/src/memory/tests_linux.rs
src/agent/onefuzz/src/blob/url.rs
src/agent/onefuzz/src/blob/client.rs
src/agent/onefuzz-task/src/tasks/stats/afl.rs
src/agent/debugger/src/module.rs
src/agent/reqwest-retry/src/lib.rs
src/agent/coverage-legacy/src/block/linux.rs
... and 189 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@AdamL-Microsoft AdamL-Microsoft left a comment

Choose a reason for hiding this comment

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

Just noticed the semantic version error, think this needs to be 111.25.0

@Porges
Copy link
Member

Porges commented Feb 7, 2023

Just noticed the semantic version error, think this needs to be 111.25.0

The checksum will need to change as well; use cargo to do the update: cargo update -p openssl-src

@Porges Porges enabled auto-merge (squash) February 7, 2023 20:35
@Porges Porges merged commit 2ab07ef into microsoft:main Feb 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants