Skip to content

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

Merged
beeme1mr merged 1 commit into
open-feature:mainfrom
Totodore:chore-bump-flagsmith-sdk
Aug 31, 2026
Merged

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

Conversation

@Totodore

@Totodore Totodore commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

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.

@jbovet

jbovet commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

@matthewelwell @Zaimwa9 @beeme1mr whenever you have a chance, could you please take a look at this PR? All checks are passing, and it's ready for review.

@beeme1mr
beeme1mr merged commit 568e0bc into open-feature:main Aug 31, 2026
6 checks passed
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.

5 participants