Skip to content

Cargo: update ring 0.17.8 -> 0.17.14#554

Merged
cpu merged 1 commit intorustls:mainfrom
cpu:cpu-update-ring
Mar 25, 2025
Merged

Cargo: update ring 0.17.8 -> 0.17.14#554
cpu merged 1 commit intorustls:mainfrom
cpu:cpu-update-ring

Conversation

@cpu
Copy link
Copy Markdown
Member

@cpu cpu commented Mar 25, 2025

Updates the Cargo.lock for *ring* to clear RUSTSEC-2025-0009

I don't believe it to be an issue that affects rustls-ffi in practice because:

  1. ring is optional, and not the default crypto provider
  2. there's no exposed API surface that would lead to ring::aead::quic usage
  3. for ring::aead::{AES_128_GCM, AES_256_GCM}, the TLS protocol limits prevent the volume of data required to hit the panic

Updates the Cargo.lock for `*ring*` to resolve RUSTSEC-2025-0009
@cpu
Copy link
Copy Markdown
Member Author

cpu commented Mar 25, 2025

I can't see why Dependabot skipped this one 🤔 That might be worth some further debugging.

@djc
Copy link
Copy Markdown
Member

djc commented Mar 25, 2025

Perhaps we should have a cargo-deny run here, too?

@cpu
Copy link
Copy Markdown
Member Author

cpu commented Mar 25, 2025

Perhaps we should have a cargo-deny run here, too?

I'm onboard. I think ~only webpki has that today. Probably worth looking at both rustls/rustls and this repo.

@cpu cpu merged commit f294692 into rustls:main Mar 25, 2025
46 checks passed
@kpcyrd
Copy link
Copy Markdown
Contributor

kpcyrd commented Mar 25, 2025

FWIW, this is what I use to invoke cargo-deny, takes about 24s in total.

You're very welcome to copy/use/adapt this.

  deny:
    runs-on: ubuntu-24.04
    steps:
    - uses: actions/checkout@v4
      with:
        persist-credentials: false
    - name: Run cargo deny
      run: |
        docker run --rm -v "$PWD:/src" -w /src alpine:edge sh -c '
        set -e
        apk add cargo cargo-deny
        exec cargo deny check
        '

@djc
Copy link
Copy Markdown
Member

djc commented Mar 26, 2025

I usually use the first-party GitHub Action.

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.

3 participants