Support node-redis 4.x.x

[luismiramirez]

Background

As reported in: https://github.com/appsignal/support/issues/184, before @appsignal/nodejs@2.3.1, users with redis v4.x.x package in their apps may experiment crashes when starting up and/or other related issues.

The problem

This was caused due to the way the version check was done for the Redis instrumentation. Here's the PR that fixes it: #566

Adapting the instrumentation to version 4.x.x is not a simple task. The node-redis package has completely changed its structure and interface. Now it doesn't export the RedisClient module, which was needed to wrap the internal_send_command function to add our instrumentation with shimmer.

The proposal(s)

Now that we're planning to keep moving to an OpenTelemetry oriented solution, I think it's best to give support to the Redis instrumentation using OpenTelemetry Redis plugin

[tombruijn]

Moving this to backlog while we wait for opentelemetry research results. https://github.com/appsignal/integration-guide/issues/47

[v-ruiz]

Thx 🙏🙏🙏

[jimaek]

We would like to see support for it as well 👍

[tombruijn]

Related #366

[tombruijn]

OpenTelemetry doesn't support version 4 yet either.

^2.6.0 || ^3.0.0 (version 4 is not yet supported)

[luismiramirez]

Supported in v3