beezz/ddtrace-graphql

Python library for Datadog tracing of graphql calls.

ddtrace-graphql's Issues

Use with Graphene

Is it possible to use this library with Graphene?

I tried using the patch function at my Django entrypoint as suggested in the docs, but the service never appeared. Is this likely to be because I am using Graphene?

My code in my wsgi.py entrypoint file is as follows:

    from ddtrace import tracer, patch_all
    from ddtrace_graphql import patch as patch_graphql

    # Connect the APM to the agent
    try:
        tracer.configure(
            hostname=os.environ['DD_AGENT_HOST']
        )
    except Exception as e:
        print('Failed to register datadog tracing host', e)

    # Activate the APM
    patch_all()

    # Support frameworks
    patch_all(botocore=True, boto=True, kombu=True, celery=True)
    patch_graphql()

Out of date with ddtrace library

When installing any version of ddtrace>=1.0.0, 0.2.0 of this library will fail on load because ddtrace has been updated. Specifically when trying to do from ddtrace.contrib.util import requires_modules, will fail because requires_module was moved in version 1.0.0 of ddtrace. So the line needs to be updated to from ddtrace.contrib import require_modules.

Other import lines to be updated are:
from ddtrace.ext import errors as ddtrace_errors -> from ddtrace.constants import ERROR_MSG, ERROR_STACK, ERROR_TYPE
from ddtrace.util import unwrap -> from ddtrace.internal.utils.wrappers import unwrap
Remove import graphql.backend.core

No data is received on ddtrace 0.55.2

I'm not sure if this library is still maintained, but we just upgraded our ddtrace (that we use for our own tracing) from 0.50.0 -> 0.55.2 and our GraphQL data stopped:

I noticed this library is using an earlier version.

Is it possible to upgrade the version used by this library? Could there be another issue?

Initial Update

Hi 👊

This is my first visit to this fine repo, but it seems you have been working hard to keep all dependencies updated so far.

Once you have closed this issue, I'll create separate pull requests for every update as soon as I find one.

That's it for now!

Happy merging! 🤖

Not picking up DD_VERSION environment variable

Hello, thanks for making this library, it's super helpful.

I'm using version 0.2.0, and I have a problem that the graphql APM service is not picking up DD_VERSION environment variable.

Any help will be really appreciated, thank you.