Skip to content

cargo: upgrade jsonwebtoken#689

Merged
tisonkun merged 1 commit intoapache:mainfrom
xnox:jsonwebtoken-upgrade
Feb 25, 2026
Merged

cargo: upgrade jsonwebtoken#689
tisonkun merged 1 commit intoapache:mainfrom
xnox:jsonwebtoken-upgrade

Conversation

@xnox
Copy link
Contributor

@xnox xnox commented Feb 14, 2026

Move jsonwebtoken to a workspace dependency.
Upgrade to 10 series, to pick up CVE fix in 10.3.0.
Use aws-lc backend, as aws-lc is already an existing transitive
dependency.

Move jsonwebtoken to a workspace dependency.
Upgrade to 10 series, to pick up CVE fix in 10.3.0.
Use aws-lc backend, as aws-lc is already an existing transitive
dependency.
mwiewior added a commit to biodatageeks/polars-bio that referenced this pull request Feb 19, 2026
Bump Rust toolchain from 1.86.0 to 1.88.0 to allow updating `time`
crate. Update Cargo.lock (`bytes` 1.11.0→1.11.1, `time` 0.3.36→0.3.47)
and poetry.lock (`pypdf`→6.7.1, `pillow`→12.1.1, `nbconvert`→7.17.0,
`weasyprint`→68.1) to resolve 9 of 10 open Dependabot alerts.

The remaining `jsonwebtoken` alert (CVE-2026-25537) cannot be fixed yet
as it requires a major version bump (9.x→10.3.0) blocked by upstream
`reqsign`/`opendal` (tracked in apache/opendal-reqsign#689).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
mwiewior added a commit to biodatageeks/polars-bio that referenced this pull request Feb 19, 2026
…316)

Bump Rust toolchain from 1.86.0 to 1.88.0 to allow updating `time`
crate. Update Cargo.lock (`bytes` 1.11.0→1.11.1, `time` 0.3.36→0.3.47)
and poetry.lock (`pypdf`→6.7.1, `pillow`→12.1.1, `nbconvert`→7.17.0,
`weasyprint`→68.1) to resolve 9 of 10 open Dependabot alerts.

The remaining `jsonwebtoken` alert (CVE-2026-25537) cannot be fixed yet
as it requires a major version bump (9.x→10.3.0) blocked by upstream
`reqsign`/`opendal` (tracked in apache/opendal-reqsign#689).

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@Xuanwo Xuanwo requested a review from tisonkun February 23, 2026 06:26
hmac = "0.12"
http = "1"
jiff = "0.2"
jsonwebtoken = { version = "10", features = ["aws_lc_rs"] }
Copy link
Member

Choose a reason for hiding this comment

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

Will this always enable aws_lc_rs? Our downstream users, such as opendal, want to allow users to choose their own cryptography implementation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

there is no choice of cryptography implementations; and other dependencies of opendal-reqsign already require aws_lc_rs. Thus this is no change in dependencies.

Copy link
Contributor Author

@xnox xnox Feb 23, 2026

Choose a reason for hiding this comment

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

Note this dependency has no affect on rustls backend in use.

Copy link
Member

@tisonkun tisonkun Feb 25, 2026

Choose a reason for hiding this comment

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

and other dependencies of opendal-reqsign already require aws_lc_rs

Yes. This is mainly pulled in from reqwest 0.13's default-tls.

We may have a follow-up to investigate whether we can make it optional (switchable between aws-lc-rs and ring).

I personally don't like aws-lc-rs too much since it pulls in cmake and other build hazards. But it's the trend, and I don't find too many motivations to build a replacement now.

Copy link
Member

@tisonkun tisonkun left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @xnox!

LGTM.

@tisonkun tisonkun merged commit efbcf35 into apache:main Feb 25, 2026
47 checks passed
C0DE-X pushed a commit to C0DE-X/os that referenced this pull request Mar 2, 2026
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