Skip to content

update: Bump reqwest from 0.12.23 to 0.13.1#1755

Open
ok-nick wants to merge 9 commits intomainfrom
ok-nick/update-reqwest
Open

update: Bump reqwest from 0.12.23 to 0.13.1#1755
ok-nick wants to merge 9 commits intomainfrom
ok-nick/update-reqwest

Conversation

@ok-nick
Copy link
Contributor

@ok-nick ok-nick commented Jan 20, 2026

Update reqwest to v0.13.1 and use default features. This fixes an error on dependabot since reqwest changed their features.
Screenshot 2026-01-16 at 5 11 32 PM

@ok-nick ok-nick added the check-release Add this label to any PR to invoke a larger suite of tests. label Jan 20, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 20, 2026

Merging this PR will not alter performance

✅ 16 untouched benchmarks
⏩ 2 skipped benchmarks1


Comparing ok-nick/update-reqwest (ae54246) with main (2f33f6c)

Open in CodSpeed

Footnotes

  1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@ok-nick ok-nick force-pushed the ok-nick/update-reqwest branch from 6886143 to b8d71da Compare January 20, 2026 21:43
Copy link
Collaborator

@scouten-adobe scouten-adobe left a comment

Choose a reason for hiding this comment

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

Please hold on merging this until we have a legal review on OpenSSL license.

Copy link
Collaborator

@scouten-adobe scouten-adobe left a comment

Choose a reason for hiding this comment

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

@ok-nick please update the deny.toml file to include this license. Otherwise, good to go!

Copy link
Collaborator

@scouten-adobe scouten-adobe left a comment

Choose a reason for hiding this comment

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

@ok-nick per your comment in Slack, looks like this may add significantly to code size. Let's hold off on merging until we can research this further.

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

Labels

check-release Add this label to any PR to invoke a larger suite of tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants