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

Data error #18

Closed
temp opened this issue Feb 13, 2023 · 7 comments
Closed

Data error #18

temp opened this issue Feb 13, 2023 · 7 comments
Assignees

Comments

@temp
Copy link

temp commented Feb 13, 2023

One of the services in our stack submits a sentry event without gz-compression, but since it is not a service from our end, we can't simply enable the gz-compression.
Version is latest dev.
I guess it should work without gz-compression, since this is plain http?

2023-02-13T11:19:16.732Z	ERROR	app         	ErrorException: fwrite(): zlib: data error in /app/vendor/clue/stream-filter/src/functions.php at line 280 [] []
@butschster
Copy link
Member

Hi!
I'll try to check it. Probably there is a way to detect is a request payload was gzipped or not.

@butschster butschster added the bug Something isn't working label Feb 13, 2023
@butschster butschster self-assigned this Feb 13, 2023
@butschster
Copy link
Member

Could you check zlib extension in your PHP?

@temp
Copy link
Author

temp commented Feb 16, 2023

The service that is sending the sentry-event is not a php-service, it's javascript (graphql). And since it is not a service that is under our control we cannot change the way it sends the sentry-event (without gz compression).
Buggregator is running via your docker image from docker hub...

@butschster
Copy link
Member

The service that is sending the sentry-event is not a php-service, it's javascript (graphql). And since it is not a service that is under our control we cannot change the way it sends the sentry-event (without gz compression). Buggregator is running via your docker image from docker hub...

Oh, now I got it. Thank you for the response! I'll try to reproduce you case and find out some solution!

@eXaminator
Copy link

We just came across the same problem, but in our case it is a react app that sends sentry issues directly via browser. Initially we got a 404 because the Sentry API eventhandler didn't find the necessary headers. After faking the headers we got the above error (obviously).

It would be awesome if buggregator could support Sentry events from JavaScript SDKs. :)

@butschster
Copy link
Member

We just came across the same problem, but in our case it is a react app that sends sentry issues directly via browser. Initially we got a 404 because the Sentry API eventhandler didn't find the necessary headers. After faking the headers we got the above error (obviously).

It would be awesome if buggregator could support Sentry events from JavaScript SDKs. :)

Hey @eXaminator,

Thanks for the suggestion! I've gone ahead and created a PR #73 to address this. I'll be rolling out a new Buggregator release soon. Stay tuned!

@butschster
Copy link
Member

Hi @temp

I've made some tweaks to Buggregator, and I believe I've resolved an issue with Sentry. Could you please take a look and verify it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants