search

Treblle / treblle-laravel

The official Treblle SDK for Laravel. Seamlessly integrate Treblle to manage communication with your dashboard, send errors, and secure sensitive data.
https://www.treblle.com/
MIT License
129 stars 21 forks source link

Don't make `TreblleMiddleware` final #56

Closed jbrooksuk closed 1 year ago

jbrooksuk commented 1 year ago

This allows us to extend the middleware. In my case, I need to reduce the amount of data being sent to Treblle.

My plan is to use the Lottery class to give some odds as to whether the data is sent or not.

JustSteveKing commented 1 year ago

Hey @jbrooksuk typically we wouldn't want others to play around and extend the middleware as the payload needs to be specific and the way the request is constructed is important - however considering who is asking I will merge this. Use with caution though please as we cannot guarantee results of extended classes 😃