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

chore(deps): update package dependencies #6447

Merged
merged 4 commits into from
Feb 17, 2021

Conversation

StephenWakely
Copy link
Contributor

@StephenWakely StephenWakely commented Feb 15, 2021

There is a security advisory on rand_core.

I have run cargo update on the project to update the dependencies to a patched version (0.6.2).

Note, a number of other crates have been updated too. I'm not particularly sure if this is a problem or not, so have decided to keep these updates.

╰─$ cargo update
    Updating git repository `https://github.com/hyperium/hyper`
    Updating crates.io index
    Updating git repository `https://github.com/bytecodealliance/lucet.git`
    Updating git repository `https://github.com/kyren/rlua`
    Updating git repository `https://github.com/tower-rs/tower`
    Updating git repository `https://github.com/tokio-rs/tracing`
    Updating async-graphql-parser v2.4.6 -> v2.5.4
    Updating bson v1.1.0 -> v1.2.0
    Updating env_logger v0.8.2 -> v0.8.3
    Updating lexical-core v0.7.4 -> v0.7.5
    Updating lru v0.6.4 -> v0.6.5
    Updating maxminddb v0.17.1 -> v0.17.2
    Updating nb-connect v1.0.2 -> v1.0.3
    Removing nom v5.1.2
    Removing nom v6.1.0
      Adding nom v6.1.2
    Updating parking_lot_core v0.8.2 -> v0.8.3
    Updating pem v0.8.2 -> v0.8.3
      Adding prost v0.7.0
      Adding prost-build v0.7.0
      Adding prost-derive v0.7.0
      Adding prost-types v0.7.0
    Updating pulsar v1.0.1 -> v1.1.0
    Updating quote v1.0.8 -> v1.0.9
    Updating rand_core v0.6.1 -> v0.6.2
    Updating redox_syscall v0.2.4 -> v0.2.5
    Updating serde_yaml v0.8.16 -> v0.8.17
    Updating tap v1.0.0 -> v1.0.1
      Adding which v4.0.2

@StephenWakely StephenWakely requested review from a team and ktff and removed request for a team February 15, 2021 17:59
Signed-off-by: Stephen Wakely <fungus.humungus@gmail.com>
@StephenWakely StephenWakely force-pushed the cargo_update_for_rand_core branch from 6291abb to ad8f53d Compare February 15, 2021 18:05
@binarylogic
Copy link
Contributor

It's fine, note how @fanatid has previously updated dependencies:#6217

Copy link
Contributor

@ktff ktff left a comment

Choose a reason for hiding this comment

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

The failing test is unrelated, and one bench needs to be updated but it seams to be unrelated to this PR, so nothing blocking, but it would be nice to update the bench and confirm that there aren't any performance regressions.

@lucperkins
Copy link
Contributor

FYI #6449 should fix that unrelated failure

…nd_core

Signed-off-by: Jesse Szwedko <jesse@szwedko.me>
@jszwedko
Copy link
Member

I merged master to run benches.

@StephenWakely StephenWakely merged commit add0934 into master Feb 17, 2021
@StephenWakely StephenWakely deleted the cargo_update_for_rand_core branch February 17, 2021 00:08
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.

5 participants