Skip to content

Commit 680074d

Browse files
committed
minor #30896 [HttpClient] Fix Notice on HttpClientTestCase::testTimeoutOnStream() (derrabus)
This PR was merged into the 4.3-dev branch. Discussion ---------- [HttpClient] Fix Notice on HttpClientTestCase::testTimeoutOnStream() | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | N/A | License | MIT | Doc PR | N/A When I run the test suite for the HTTP Client locally, I run into the following notice: ``` Notice: ob_flush(): failed to flush buffer. No buffer to flush in /path/to/symfony/src/Symfony/Contracts/HttpClient/Test/Fixtures/web/index.php on line 89 ``` This PR aims to fix this. `EUFOSSA` Commits ------- 8acf29e There is no OB to flush.
2 parents e53f696 + 8acf29e commit 680074d

File tree

1 file changed

+2
-2
lines changed
  • src/Symfony/Contracts/HttpClient/Test/Fixtures/web

1 file changed

+2
-2
lines changed

src/Symfony/Contracts/HttpClient/Test/Fixtures/web/index.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@
8686

8787
case '/timeout-body':
8888
echo '<1>';
89-
ob_flush();
89+
@ob_flush();
9090
flush();
9191
usleep(500000);
9292
echo '<2>';
@@ -97,7 +97,7 @@
9797
sleep(1);
9898
while (true) {
9999
echo '<1>';
100-
ob_flush();
100+
@ob_flush();
101101
flush();
102102
usleep(500);
103103
}

0 commit comments

Comments
 (0)