bug(auth): Prevent double log scenario

Because: - Logs were being emitted twice This Commit: - Provides a name to the logger instance sent into node-tracing
mozilla · Sep 30, 2022 · 301eb89 · 301eb89
1 parent 7c65917
commit 301eb89
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/packages/fxa-auth-server/bin/key_server.js b/packages/fxa-auth-server/bin/key_server.js
@@ -10,7 +10,7 @@ const config = require('../config');
 const TracingProvider = require('fxa-shared/tracing/node-tracing');
 TracingProvider.init(
   config.get('tracing'),
-  require('../lib/log')({ ...config.log })
+  require('../lib/log')({ ...config.log, name: 'tracing' })
 );
 
 const error = require('../lib/error');

diff --git a/packages/fxa-auth-server/lib/log.js b/packages/fxa-auth-server/lib/log.js
@@ -20,6 +20,7 @@ const CLIENT_ID_TO_SERVICE_NAMES = config.get('oauth.clientIds') || {};
 function Lug(options) {
   EventEmitter.call(this);
   this.name = options.name || 'fxa-auth-server';
+
   this.logger = mozlog({
     app: this.name,
     level: options.level,