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

Predis Integration exception #315

Closed
ynuyasha opened this issue Feb 21, 2019 · 2 comments
Closed

Predis Integration exception #315

ynuyasha opened this issue Feb 21, 2019 · 2 comments
Labels
🐛 bug Something isn't working help wanted Extra attention is needed 🎉 new-integration A new integration 🤲 up-for-grabs Anyone who wants to contribute to the project can take this on

Comments

@ynuyasha
Copy link

Hi,
if i enable ddtrace module, my application breaks with this error:

Recoverable Error: Object of class Predis\Connection\Aggregate\MasterSlaveReplication could not be converted to string (File: /opt/datadog-php/dd-trace-sources/src/DDTrace/Integrations/Predis/PredisIntegration.php - Line: 171)

@labbati labbati added 🐛 bug Something isn't working help wanted Extra attention is needed 🎉 new-integration A new integration 🤲 up-for-grabs Anyone who wants to contribute to the project can take this on labels Feb 21, 2019
@labbati
Copy link
Member

labbati commented Feb 21, 2019

Hi @ynuyasha 👋 , thanks for reporting this. Not sure if you are interested in it, but if you like to contribute this would be a very good first issue. Otherwise, we will work on it as soon as possible.

In the meantime, you can still use all the other integrations and disable only Predis DD_INTEGRATIONS_DISABLED=predis (see docs)

@bwoebi
Copy link
Collaborator

bwoebi commented Jan 26, 2022

This has been fully fixed by #574 and then as part of #1094.

@bwoebi bwoebi closed this as completed Jan 26, 2022
estringana added a commit that referenced this issue Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working help wanted Extra attention is needed 🎉 new-integration A new integration 🤲 up-for-grabs Anyone who wants to contribute to the project can take this on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants