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

Cargo audit fix #512

Merged
merged 12 commits into from
Jan 10, 2022
Merged

Cargo audit fix #512

merged 12 commits into from
Jan 10, 2022

Conversation

koskja
Copy link
Contributor

@koskja koskja commented Jan 8, 2022

Get rid of cargo-audit issues

Status

  • Ready
  • Development
  • Hold

Description

I have shamelessly stolen @noahcoetsee 's work and finished it to the best of my abilities by running cargo-update to bump wasmer-wasi to version 2.1.1, which gets rid of the time dependency.

Related issues

#506 - original PR

Checklist

  • Ran cargo fmt, cargo clippy --all-targets, cargo build --release and cargo test and fixed any generated errors!
  • Removed unnecessary commented out code
  • Used specific traces (if you trace actions please specify the cause i.e. the player)

noahcoetsee and others added 10 commits November 6, 2021 23:11
… it has addressed the issues present in RUSTSEC-2020-0159 in its own security advisory: RUSTSEC-2020-071.
…ing` module that mimics the `feather-server` setup.
Note that this still leaves the `rsa` issue until the `pem-rfc7468`/`pkcs8` dependency issues are resolved within that crate.
…account for new `RsaPrivateKey` capitalization.
Added `host-fs` and `sys` to `wasmer-wasi` dependencies
Removed `time` from the list of `zip` dependencies
@noahcoetsee
Copy link
Contributor

Happy to see this finally got done 🤣

feather/server/Cargo.toml Outdated Show resolved Hide resolved
feather/server/src/logging.rs Show resolved Hide resolved
tools/proxy/Cargo.toml Outdated Show resolved Hide resolved
tools/proxy/src/logging.rs Show resolved Hide resolved
@koskja
Copy link
Contributor Author

koskja commented Jan 8, 2022

Feels done.

@Defman Defman merged commit 57ca709 into feather-rs:main Jan 10, 2022
@Schuwi
Copy link
Member

Schuwi commented Jan 11, 2022

Why does this PR activate new features for wasmer-wasi and zip?

@koskja
Copy link
Contributor Author

koskja commented Jan 11, 2022

Zip actually only gets the time feature removed and wasmer-wasi needs these features selected after an update to 2.1.1

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