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

Don't crash when using mongodb driver 2.0.16 #194

Closed
wants to merge 1 commit into from
Closed

Don't crash when using mongodb driver 2.0.16 #194

wants to merge 1 commit into from

Conversation

vkarpov15
Copy link

One of the companies that's been helping QA the new version of the mongodb driver uses New Relic in their test environment, and turns out the New Relic agent crashes ungracefully with the new version of the driver. See Automattic/mongoose#2652 and mongodb/node-mongodb-native#1253. We added the ability to get the collection name from the cursor back into the driver for v2.0.16. I'm not sure if this is the correct way to handle this on your end, but this tweak seems to be enough to at least make the agent stop crashing.

@vkarpov15
Copy link
Author

Actually, woops, didn't look at the PRs and so didn't see discussion on #193. Feel free to close if you deem it necessary. But hopefully the fact that you can get collection name from cursor helps :)

@hayes
Copy link
Contributor

hayes commented Feb 18, 2015

Thanks for looking in to this, we have an updated version of our mongo instrumentation that we are releasing very soon that supports getting collection names across various versions of mognodb. I'll add a note here when its released.

@hayes hayes closed this Feb 18, 2015
@vkarpov15
Copy link
Author

Much thanks, I'm looking forward to the release. In the meantime, feel free to report any issues you run into on our JIRA board.

@hayes
Copy link
Contributor

hayes commented Feb 26, 2015

a fix for this is released in 1.17.0

@vkarpov15
Copy link
Author

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants