Skip to content

Replaced php-http/message-factory with psr/http-factory and psr/http-client#45

Open
evertharmeling wants to merge 3 commits into
bresam:masterfrom
evertharmeling:psr-refactor
Open

Replaced php-http/message-factory with psr/http-factory and psr/http-client#45
evertharmeling wants to merge 3 commits into
bresam:masterfrom
evertharmeling:psr-refactor

Conversation

@evertharmeling
Copy link
Copy Markdown

@evertharmeling evertharmeling commented Sep 8, 2025

Fixes the Package php-http/message-factory is abandoned, you should avoid using it. Use psr/http-factory instead. message.

Changes:

  • Replaced Http\Message\MessageFactory with Psr\Http\Message\RequestFactoryInterface
  • Replaced Http\Client\HttpClient with Psr\Http\Client\ClientInterface
  • Dropped references to Httplug as it's not needed anymore with full PSR-17 and PSR-18 compatibility, also simplifies configuration.

When merged and a new release is available I will update the version requirement in bresam/ivory-google-map-bundle#33.

All feedback is welcome :)

@evertharmeling evertharmeling changed the title Replaced php-http/message-factory with psr/http-factory and psr/http-client [WIP] Replaced php-http/message-factory with psr/http-factory and psr/http-client Sep 8, 2025
@evertharmeling evertharmeling marked this pull request as draft September 8, 2025 13:23
@evertharmeling evertharmeling marked this pull request as ready for review September 8, 2025 14:26
@evertharmeling evertharmeling changed the title [WIP] Replaced php-http/message-factory with psr/http-factory and psr/http-client Replaced php-http/message-factory with psr/http-factory and psr/http-client Sep 8, 2025
@evertharmeling evertharmeling force-pushed the psr-refactor branch 4 times, most recently from b4b06be to 40de1e4 Compare September 8, 2025 19:34
@evertharmeling
Copy link
Copy Markdown
Author

@bresam any chance you could take a look at this PR?

Comment thread .github/workflows/build.yml Outdated
@evertharmeling evertharmeling force-pushed the psr-refactor branch 2 times, most recently from 125371b to cba32da Compare November 4, 2025 10:04
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.

2 participants