Skip to content

chore: update guzzle-cache-middleware from 4.x to 7.0 - #18

Merged
kinyoklion merged 1 commit into
mainfrom
devin/1779836117-update-guzzle-cache
May 27, 2026
Merged

chore: update guzzle-cache-middleware from 4.x to 7.0#18
kinyoklion merged 1 commit into
mainfrom
devin/1779836117-update-guzzle-cache

Conversation

@kinyoklion

Copy link
Copy Markdown
Member

Summary

Update kevinrob/guzzle-cache-middleware from 4.* to ^7.0 to fix PHP 8.4+ deprecation warnings about implicit nullable parameters.

The LD PHP SDK uses guzzle-cache-middleware via GuzzleFeatureRequester for HTTP response caching in polling mode. The v7 API is backward-compatible — CacheMiddleware, PublicCacheStrategy, and Psr6CacheStorage all retain the same interfaces.

Review & Testing Checklist for Human

  • Run composer install && php main.php and verify no deprecation warnings appear
  • Verify flag evaluation still works correctly

Notes

Tested with PHP 8.4 via Docker — no deprecation warnings with v7, app evaluates flags correctly.

Link to Devin session: https://app.devin.ai/sessions/b648d8cd5e0b4a04ba0cbf1dcd1a9886
Requested by: @kinyoklion

Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration Bot added the devin-pr PRs created by Devin label May 26, 2026
@kinyoklion
kinyoklion marked this pull request as ready for review May 26, 2026 22:57
@kinyoklion
kinyoklion requested a review from a team as a code owner May 26, 2026 22:57
@kinyoklion
kinyoklion merged commit 9b54024 into main May 27, 2026
4 checks passed
@kinyoklion
kinyoklion deleted the devin/1779836117-update-guzzle-cache branch May 27, 2026 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devin-pr PRs created by Devin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants