Skip to content

chore(deps): bump flagsmith sdk to 3.0 - #157

Open
Totodore wants to merge 1 commit into
open-feature:mainfrom
Totodore:chore-bump-flagsmith-sdk
Open

chore(deps): bump flagsmith sdk to 3.0#157
Totodore wants to merge 1 commit into
open-feature:mainfrom
Totodore:chore-bump-flagsmith-sdk

Conversation

@Totodore

@Totodore Totodore commented Aug 11, 2026

Copy link
Copy Markdown

This PR

Bump the flagsmith sdk to its latest version 3.0.0.
This most notably allows depends on the latest version of reqwest, the main motivation can be found here: Flagsmith/flagsmith-rust-client#49

Notes

There is no need to actually pull the reqwest crate ourselve for the headermap as it is a re-export of the http crate which is stable.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The Flagsmith SDK dependency is upgraded from version 2.1 to 3.0. The crate replaces reqwest with http for custom header types. An unused test import is removed.

Changes

Flagsmith HTTP dependency update

Layer / File(s) Summary
Dependency and header contract
crates/flagsmith/Cargo.toml, crates/flagsmith/src/lib.rs
The Flagsmith dependency is upgraded to 3.0. reqwest is replaced by http version 1. FlagsmithOptions::custom_headers now uses http::HeaderMap.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: beeme1mr

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly and concisely identifies the Flagsmith SDK dependency upgrade to version 3.0.
Description check ✅ Passed The description explains the Flagsmith SDK upgrade and the related switch from reqwest headers to the http crate.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Signed-off-by: tprevot <tprevot@centreon.com>
@Totodore
Totodore force-pushed the chore-bump-flagsmith-sdk branch from f26627b to b1ffba4 Compare August 11, 2026 08:55

@jbovet jbovet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

4 participants