Merge pull request #18 from MetinSeylan/hotfix/guard-fn-scope

hotfix for guard injector scope
MetinSeylan · Nov 20, 2021 · 03696f7 · 03696f7
2 parents d22a890 + 98e32ed
commit 03696f7
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 7 deletions.
diff --git a/dist/Trace/Injectors/GuardInjector.js b/dist/Trace/Injectors/GuardInjector.js
diff --git a/dist/Trace/Injectors/GuardInjector.js.map b/dist/Trace/Injectors/GuardInjector.js.map
diff --git a/dist/tsconfig.build.tsbuildinfo b/dist/tsconfig.build.tsbuildinfo
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@metinseylan/nestjs-opentelemetry",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "deeply integrated OpenTelemetry module for Nestjs",
   "author": "[email protected]",
   "license": "MIT",

diff --git a/src/Trace/Injectors/GuardInjector.ts b/src/Trace/Injectors/GuardInjector.ts
@@ -20,11 +20,12 @@ export class GuardInjector extends BaseTraceInjector implements Injector {
         const guards = this.getGuards(controller.metatype).map((guard) => {
           const prototype = guard['prototype'] ?? guard;
           const traceName = `Guard->${controller.name}.${prototype.constructor.name}`;
-          guard.canActivate = this.wrap(prototype.canActivate, traceName, {
+          prototype.canActivate = this.wrap(prototype.canActivate, traceName, {
             controller: controller.name,
             guard: prototype.constructor.name,
             scope: 'CONTROLLER',
           });
+          Object.assign(prototype, this);
           this.loggerService.log(`Mapped ${traceName}`, this.constructor.name);
           return guard;
         });
@@ -44,12 +45,13 @@ export class GuardInjector extends BaseTraceInjector implements Injector {
             (guard) => {
               const prototype = guard['prototype'] ?? guard;
               const traceName = `Guard->${controller.name}.${controller.metatype.prototype[key].name}.${prototype.constructor.name}`;
-              guard.canActivate = this.wrap(prototype.canActivate, traceName, {
+              prototype.canActivate = this.wrap(prototype.canActivate, traceName, {
                 controller: controller.name,
                 guard: prototype.constructor.name,
                 method: controller.metatype.prototype[key].name,
                 scope: 'CONTROLLER_METHOD',
               });
+              Object.assign(prototype, this);
               this.loggerService.log(
                 `Mapped ${traceName}`,
                 this.constructor.name,
@@ -90,6 +92,7 @@ export class GuardInjector extends BaseTraceInjector implements Injector {
             scope: 'GLOBAL',
           },
         );
+        Object.assign(provider.metatype.prototype, this);
         this.loggerService.log(`Mapped ${traceName}`, this.constructor.name);
       }
     }