Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: ob_flush(): Failed to flush buffer. No buffer to flush #35

Closed
edalzell opened this issue Aug 11, 2023 · 7 comments
Closed

[Bug]: ob_flush(): Failed to flush buffer. No buffer to flush #35

edalzell opened this issue Aug 11, 2023 · 7 comments
Labels
bug Something isn't working

Comments

@edalzell
Copy link

edalzell commented Aug 11, 2023

What happened?

Running SSE normally (redis worker running, Echo on front end).

See a ton of these in my log: https://flareapp.io/share/87newjDP

How to reproduce the bug

Run my app.

Sorry don't know the repro issue yet.

Package Version

0.7.1

PHP Version

8.2.8

Laravel Version

10.18.0

Which operating systems does with happen with?

macOS

Notes

No response

@edalzell edalzell added the bug Something isn't working label Aug 11, 2023
@edalzell
Copy link
Author

This code silences the error, not sure yet if everything still works:

if (ob_get_contents()) {
    ob_flush();
}

@slavarazum
Copy link
Contributor

Hi @edalzell. Probably you have disabled output buffering on the server.

Is it works normal on a local environment?
Have you debugged what echo $this returns at ServerSentEvent::__invoke method?

@edalzell
Copy link
Author

edalzell commented Aug 15, 2023

Hi @edalzell. Probably you have disabled output buffering on the server.

Is it works normal on a local environment? Have you debugged what echo $this returns at ServerSentEvent::__invoke method?

This error is occurring locally. And I didn't get it before, it's a new thing.

@edalzell
Copy link
Author

But I did switch recently to Herd (from Brew/Valet), so maybe it is off? I'll check, thanks!

@slavarazum
Copy link
Contributor

Probably we have found a source, let test it on Herd.

@edalzell
Copy link
Author

edalzell commented Aug 22, 2023

Confirmed, Herd does NOT have output_buffering = 4096 in its php.ini.

Sorry about the confusion.

@slavarazum
Copy link
Contributor

NP, thank you for your involvement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants