fix(instrumentation-redis-4): avoid shimmer warning by only wrapping …

…multi/MULTI if they exist (#1729)

Co-authored-by: Marc Pichler <[email protected]>

plugins/node/opentelemetry-instrumentation-redis-4/src/instrumentation.ts

@@ -166,23 +166,31 @@ export class RedisInstrumentation extends InstrumentationBase<any> {
         this._diag.debug('Patching redis client');
         const redisClientPrototype = moduleExports?.default?.prototype;
 
-        if (isWrapped(redisClientPrototype?.multi)) {
-          this._unwrap(redisClientPrototype, 'multi');
+        // In some @redis/client versions 'multi' is a method. In later
+        // versions, as of https://github.com/redis/node-redis/pull/2324,
+        // 'MULTI' is a method and 'multi' is a property defined in the
+        // constructor that points to 'MULTI', and therefore it will not
+        // be defined on the prototype.
+        if (redisClientPrototype?.multi) {
+          if (isWrapped(redisClientPrototype?.multi)) {
+            this._unwrap(redisClientPrototype, 'multi');
+          }
+          this._wrap(
+            redisClientPrototype,
+            'multi',
+            this._getPatchRedisClientMulti()
+          );
         }
-        this._wrap(
-          redisClientPrototype,
-          'multi',
-          this._getPatchRedisClientMulti()
-        );
-
-        if (isWrapped(redisClientPrototype?.MULTI)) {
-          this._unwrap(redisClientPrototype, 'MULTI');
+        if (redisClientPrototype?.MULTI) {
+          if (isWrapped(redisClientPrototype?.MULTI)) {
+            this._unwrap(redisClientPrototype, 'MULTI');
+          }
+          this._wrap(
+            redisClientPrototype,
+            'MULTI',
+            this._getPatchRedisClientMulti()
+          );
         }
-        this._wrap(
-          redisClientPrototype,
-          'MULTI',
-          this._getPatchRedisClientMulti()
-        );
 
         if (isWrapped(redisClientPrototype?.sendCommand)) {
           this._unwrap(redisClientPrototype, 'sendCommand');