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

Update to rollbar-php 3.x #120

Closed
xtfer opened this issue Jan 11, 2022 · 5 comments · Fixed by #129 or #131
Closed

Update to rollbar-php 3.x #120

xtfer opened this issue Jan 11, 2022 · 5 comments · Fixed by #129 or #131
Assignees

Comments

@xtfer
Copy link

xtfer commented Jan 11, 2022

I'm getting a persistent notice in rollbar about updating the library to rollbar-php 3.x. Will this project be getting an update or should I ignore it?

@jonnott
Copy link

jonnott commented Jan 12, 2022

I was thinking that package should also be prepared for the imminent release of Laravel 9.x. Maybe these could be rolled into a new major version release together. Both Laravel 9.x and rollbar-php 3.x require PHP 8.x.

Any word from the maintainers?

@bxsx @bishopb @letaniaferreira @waltjones @ArturMoczulski

@nateiorg
Copy link

nateiorg commented Feb 4, 2022

Any word?

This isn't currently as benign as it may appear. The package requires:

"php": ">=7.2",

Yet, as of 8.1.0 many Rollbar classes are spamming instrumentation with deprecation warnings, and some classes are even outright failing to resolve as is being reported by Rollbar itself:

Error: Class "Rollbar\Payload\Person"

As one can imagine, with instrumentation being priced on rates, this can indeed be costing many people money (or resulting in throttling). If this package even limited PHP to a maximum of 8.0.x, my deployments wouldn't have started using 8.1.x for containers. That realistically should be in place until 8.1.x is supported.

Can we please see an update here? I'm running Octane, which means I see a single bootstrap per deployment, but anyone who has a bootstrap per-request is going to be seeing tens of thousands of errors flooding instrumentation (or more). Even in my case, the flood of alerts post-deployment are obnoxious, and have already resulted in rate limiting.

I get that the deprecation warning is "just a warning", however, we run quite strict to combat accidental type coercion (and if I'm being honest, for this exact scenario -- we absolutely don't want to be running deprecated code). If a company is forced to choose between using Rollbar and shifting their (arguably better) current paradigm to another (arguably worse) paradigm, Rollbar loses the battle every single time. It's really not even debatable whether we should start allowing/suppressing/ignoring warnings or not -- we won't.

EDIT:

I just realized that rollbar-php 3.x doesn't even address these issues. Time to jump ship, and based on the comments within the issues in the rollbar-php repo, you may want to as well. There are just as many cobwebs in the issues section there.

@danielmorell
Copy link
Collaborator

danielmorell commented Feb 25, 2022

Hey, I just want to jump on this issue and let everyone know that I am working on this and should have this resolved soon.

@danielmorell danielmorell self-assigned this Feb 25, 2022
@xtfer
Copy link
Author

xtfer commented Feb 25, 2022

Thanks @danielmorell. Happy to be a test guinea pig if you need one/

@danielmorell
Copy link
Collaborator

This has been resolved with the release of v7.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants