Merge pull request #27 from MetinSeylan/hotfix/decorator

Hotfix observer decorators
MetinSeylan · Feb 26, 2022 · 573b10f · 573b10f
2 parents 1863c76 + f8431ec
commit 573b10f
Show file tree

Hide file tree

Showing 8 changed files with 13 additions and 4 deletions.
diff --git a/dist/Metric/Injectors/DecoratorCounterMetricInjector.js b/dist/Metric/Injectors/DecoratorCounterMetricInjector.js
diff --git a/dist/Metric/Injectors/DecoratorCounterMetricInjector.js.map b/dist/Metric/Injectors/DecoratorCounterMetricInjector.js.map
diff --git a/dist/Metric/Injectors/DecoratorObserverMetricInjector.js b/dist/Metric/Injectors/DecoratorObserverMetricInjector.js
diff --git a/dist/Metric/Injectors/DecoratorObserverMetricInjector.js.map b/dist/Metric/Injectors/DecoratorObserverMetricInjector.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.3",
+  "version": "2.0.4",
   "description": "deeply integrated OpenTelemetry module for Nestjs",
   "author": "[email protected]",
   "license": "MIT",

diff --git a/src/Metric/Injectors/DecoratorCounterMetricInjector.ts b/src/Metric/Injectors/DecoratorCounterMetricInjector.ts
@@ -41,6 +41,8 @@ export class DecoratorCounterMetricInjector extends BaseMetricInjector {
           !this.isAffected(provider.metatype.prototype[key])
         ) {
           const options = this.getOptions(provider.metatype.prototype[key]);
+          if (options.type !== DecoratorType.COUNTER) return;
+
           const name =
             options['name']?.toLowerCase() ??
             this.generateName(provider, provider.metatype.prototype[key]);

diff --git a/src/Metric/Injectors/DecoratorObserverMetricInjector.ts b/src/Metric/Injectors/DecoratorObserverMetricInjector.ts
@@ -76,6 +76,9 @@ export class DecoratorObserverMetricInjector extends BaseMetricInjector {
           const options = this.getOptions(
             isControllerDecorated ? controller.metatype : prototype,
           );
+
+          if (options.type !== DecoratorType.OBSERVER) return;
+
           const name = this.generateName(controller, prototype, options);
           const metric = this.generateMetric(name, options['options']);