Fixed wrong ltrim usage in guzzle transport (#1783)

* Fixing bug in usage of ltrim * Add changelog Co-authored-by: François-Xavier de Guillebon <deguif@gmail.com>
ruflin · Mar 24, 2021 · 5ca8cfc · 5ca8cfc
1 parent c356bac
commit 5ca8cfc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -20,6 +20,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * Deprecated `Elastica\Transport\HttpAdapter` class [#1940](https://github.com/ruflin/Elastica/pull/1940)
 ### Removed
 ### Fixed
+* Fixed wrong `ltrim` usage in guzzle transport [#1783](https://github.com/ruflin/Elastica/pull/1783)
 * Fixed `_seq_no` and `_primary_term` wrong initialization [#1920](https://github.com/ruflin/Elastica/pull/1920)
 * Fixed `Elastica\Connection\StrategyInterface` instance checks [#1921](https://github.com/ruflin/Elastica/pull/1921)
 * Fixed various PHPDoc annotations [#1922](https://github.com/ruflin/Elastica/pull/1922)

diff --git a/src/Transport/Guzzle.php b/src/Transport/Guzzle.php
@@ -168,7 +168,7 @@ protected function _getBaseUrl(Connection $connection): string
                 'scheme' => $this->_scheme,
                 'host' => $connection->getHost(),
                 'port' => $connection->getPort(),
-                'path' => \ltrim('/', $connection->getPath()),
+                'path' => \ltrim($connection->getPath(), '/'),
             ]);
         }