Please sign in to comment.
bug #29256 [HttpFoundation] Fixed absolute Request URI with default p…
…ort (thomasbisignani) This PR was merged into the 3.4 branch. Discussion ---------- [HttpFoundation] Fixed absolute Request URI with default port | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #29234 | License | MIT | Doc PR | This PR fixes the #29234 issue, the request URI with default port was not properly generated. Example : ```php $request = Request::create('http://test.com:80/foo'); $request->server->set('REQUEST_URI', 'http://test.com:80/foo'); ``` Before this fix, the `$request->getUri()` method returned `http://test.com/:80/foo` : ```diff Failed asserting that two strings are equal. --- Expected +++ Actual @@ @@ -'http://test.com/foo' +'http://test.com/:80/foo' ``` Commits ------- cddce2a [HttpFoundation] Fixed absolute Request URI with default port
- Loading branch information...
Showing with 58 additions and 3 deletions.