search

buggregator / server

Buggregator is a lightweight, standalone server that offers a range of debugging features for PHP applications.
https://buggregator.dev/
Other
648 stars 23 forks source link

Data error #18

Closed temp closed 4 months ago

temp commented 1 year ago

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 commented 1 year ago

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

butschster commented 1 year ago

Could you check zlib extension in your PHP?

temp commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 11 months ago

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 https://github.com/buggregator/server/pull/73 to address this. I'll be rolling out a new Buggregator release soon. Stay tuned!

butschster commented 9 months ago

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?