Broken custom dataloader methods

Is there an existing issue for this?

I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
I have reviewed the documentation https://docs.sentry.io/
I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/node

SDK Version

8.33.1

Framework Version

fastify 5.0.0

Link to Sentry event

No response

Reproduction Example/SDK Setup

No response

Steps to Reproduce

The automated dataloader integration added in #13664 breaks my code.

I have a dataloader like this:

export class ProductDataloader extends DataLoader {
    constructor() {
        super(async (globalIds: ReadonlyArray<string>) => {
            return batchFn(globalIds);
        });
    }

    public async loadByInternalId(id: number) {
        const globalId = toGlobalId('Product', id);
        return this.load(globalId);
    }
}

Expected Result

That code worked OK until "@sentry/node": "8.30.0"

Actual Result

After upgrade to "@sentry/node": "8.31.0" or later, calling loadByInternalId method fails:

context.dataloaders.product.loadByInternalId is not a function

@michalsanger

Hey @michalsanger sorry for the trouble here! For now you can disable the dataloader integration like so:

Sentry.init({
  // ...
  integrations: function (integrations) {
    // filter out Dataloader integration due to bug
    return integrations.filter(function (integration) {
      return integration.name !== "Dataloader";
    });
  },
});

docs: https://docs.sentry.io/platforms/javascript/guides/node/configuration/integrations/#removing-a-default-integration

@onurtemizkan could you help investigate why this is happening?

@AbhiPrasad

@AbhiPrasad thanks for the workaround, works OK.

fix(node): Remove `dataloader` instrumentation from default integrati…

@onurtemizkan

@onurtemizkan I guess this has since been released and should be fixed, since v10 where we are on latest of the instrumentation again?

@mydea

@mydea - Yes, this should be resolved

feat(node): Enable dataloader and knex integrations by default

feat(node): Enable dataloader and knex integrations by default (#24312)

Navigation Menu

Repository navigation

Broken custom dataloader methods #13869

Is there an existing issue for this?

How do you use Sentry?

Which SDK are you using?

SDK Version

Framework Version

Link to Sentry event

Reproduction Example/SDK Setup

Steps to Reproduce

Expected Result

Actual Result

2 remaining items

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Broken custom dataloader methods #13869

Description

Is there an existing issue for this?

How do you use Sentry?

Which SDK are you using?

SDK Version

Framework Version

Link to Sentry event

Reproduction Example/SDK Setup

Steps to Reproduce

Expected Result

Actual Result

Activity

AbhiPrasad commented on Oct 4, 2024

michalsanger commented on Oct 4, 2024

2 remaining items

mydea commented on Aug 11, 2025

onurtemizkan commented on Aug 11, 2025

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions