Skip to content

Issue #212: Compatibility with psr/http-message 2.0#214

Open
omarlopesino wants to merge 3 commits intoSparkPost:masterfrom
Metadrop:feature/psr-http-message-2.0
Open

Issue #212: Compatibility with psr/http-message 2.0#214
omarlopesino wants to merge 3 commits intoSparkPost:masterfrom
Metadrop:feature/psr-http-message-2.0

Conversation

@omarlopesino
Copy link

This adapts the SparkpostResponse class to be compatible with psr http-message 2.0

@phpfui
Copy link

phpfui commented Jan 9, 2024

Looks like this library has not been updated for 3 years and is now incompatible with some modern dependencies. I would say it has been abandoned by SparkPost. I have seen this before in vendor supported libraries. I had to do a completely new Constant Contact V3 API as Constant Contact stopped supporting API libraries for all languages. Also SparkPost was recently acquired by MessageBird, so I suspect this library (and SparkPost in general) will not be receiving the attention needed to keep it current.

I would be willing to fork and maintain this repo if you want to move your PR to the new fork.

@dunkoh
Copy link

dunkoh commented Feb 7, 2024

@omarlopesino what http client are you using to pass into SparkPost?

@ptwebserv
Copy link

Thanks for the merge request, Looks like abandon ware.

@omarlopesino
Copy link
Author

I haven't seen the notifications on this issue, sorry. I'll reply in the case there is a chance to update the main library yet.

@omarlopesino what http client are you using to pass into SparkPost?

IIRC I was using guzzle 7 along with guzzle 6 adapter. It was in a Drupal project using sparkpost contrib module: https://git.drupalcode.org/project/sparkpost/-/blob/8.x-2.x/src/ClientService.php?ref_type=heads#L49

@omarlopesino
Copy link
Author

There is a fork that have integrated this PR xtremepush#2 and released it, so it looks like the fixes are being applied in other projects too.

@phpfui
Copy link

phpfui commented Mar 11, 2026

Looks like Xtremepush has not published this package to Composer. I will add a request to do that, as they have done it for another package. If no response, I will clone and publish.

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