Skip to content

chore(deps): bump chacha20poly1305 from 0.10.1 to 0.11.0 in /packages#949

Open
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/cargo/packages/chacha20poly1305-0.11.0
Open

chore(deps): bump chacha20poly1305 from 0.10.1 to 0.11.0 in /packages#949
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/cargo/packages/chacha20poly1305-0.11.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jul 16, 2026

Copy link
Copy Markdown
Contributor

Bumps chacha20poly1305 from 0.10.1 to 0.11.0.

Commits

@dependabot
dependabot Bot force-pushed the dependabot/cargo/packages/chacha20poly1305-0.11.0 branch from 6522aa8 to e3823dd Compare July 16, 2026 18:17
@github-actions github-actions Bot deleted a comment from claude Bot Jul 16, 2026
Bumps [chacha20poly1305](https://github.com/RustCrypto/AEADs) from 0.10.1 to 0.11.0.
- [Commits](RustCrypto/AEADs@chacha20poly1305-v0.10.1...chacha20poly1305-v0.11.0)

---
updated-dependencies:
- dependency-name: chacha20poly1305
  dependency-version: 0.11.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot
dependabot Bot force-pushed the dependabot/cargo/packages/chacha20poly1305-0.11.0 branch from e3823dd to 097977a Compare July 17, 2026 20:25
@github-actions github-actions Bot deleted a comment from claude Bot Jul 17, 2026
@claude

claude Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Dependabot review: chacha20poly1305 0.10.1 → 0.11.0

Change: direct production dependency bump in packages/Cargo.toml (packages/Cargo.lock updated accordingly). Dependabot labels it semver-minor, but this is a pre-1.0 crate (0.10.x0.11.x), which the Rust ecosystem treats as a breaking-change boundary, not a routine minor bump. The upstream commit list is mostly workspace-wide lint fixes and a "Trusted Publishing" CI change for the RustCrypto/AEADs repo, with no explicitly called-out API break, but I could not fetch the crate's CHANGELOG.md to confirm (no web access in this session) — worth a human eyeballing the diff before trusting that.

Where it's used: packages/editor-api/src/crypto.rs — this is the AEAD cipher sealing GitHub OAuth tokens stored in a browser cookie (TokenCipher, using ChaCha20Poly1305::{new_from_slice, encrypt, decrypt} plus OsRng/Nonce). This is exactly the kind of security-critical, auth-adjacent dependency called out for extra scrutiny regardless of bump size.

CI status: still in progress at review time (Test, Pre-commit, WASM Build, and several build jobs pending); Security Audit, Validate PR title, and Detect changes have passed so far.

Decision: flagging for human review, not auto-merging — because (a) it's a crypto/auth-critical dependency and (b) CI has not yet fully completed. If CI passes clean and a maintainer confirms no breaking change in the chacha20poly1305/aead traits used here (Aead::encrypt/decrypt, KeyInit::new_from_slice), this looks safe to merge as-is — the existing unit tests in crypto.rs (roundtrip, tamper detection, wrong-key rejection, nonce uniqueness) already exercise the exact surface this bump touches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants