elastic / elasticsearch-php

Official PHP client for Elasticsearch.
https://www.elastic.co/guide/en/elasticsearch/client/php-api/current/index.html
MIT License
5.27k stars 965 forks source link

Allow installation of psr/http-message v2.0 #1313

Closed gjuric closed 1 year ago

gjuric commented 1 year ago

Related to https://github.com/elastic/elastic-transport-php/pull/17

I've signed the CLA in the meantime.

elasticmachine commented 1 year ago

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

gjuric commented 1 year ago

The tests are failing because php-http/client-common in v2.7 dropped the dependency on php-http/message-factory.

See https://github.com/getsentry/sentry-php/issues/1533 for a similar case.

ezimuel commented 1 year ago

I fixed the issue and merged in main manually. Thanks!

ezimuel commented 1 year ago

@gjuric I had to remove the tag v8.8.1 because it has been released before the official schedule. The tag v8.8.1 will be re-created soon in a few days. Sorry for any inconvenience.

gjuric commented 1 year ago

Thanks for the heads up.

ezimuel commented 1 year ago

Released v8.8.1 with #1313