Merge pull request #14431 from mozilla/train-244-uplift-3

Train 244 uplift 3
mozilla · Nov 8, 2022 · ef3ee0d · ef3ee0d
2 parents 8efe464 + f94b9e9
commit ef3ee0d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/packages/fxa-auth-server/lib/pushbox/index.ts b/packages/fxa-auth-server/lib/pushbox/index.ts
@@ -176,11 +176,7 @@ export const pushboxApi = (
             'pushbox.db.retrieve.success',
             performance.now() - startTime
           );
-          statsd.increment('pushbox.db.retrieve', {
-            uid,
-            deviceId,
-            msgCount: result.messages.length.toString(),
-          });
+          statsd.increment('pushbox.db.retrieve');
           return {
             last: result.last,
             index: result.index,
@@ -254,7 +250,7 @@ export const pushboxApi = (
             'pushbox.db.store.success',
             performance.now() - startTime
           );
-          statsd.increment('pushbox.db.store', { uid, deviceId });
+          statsd.increment('pushbox.db.store');
           return { index: result.idx };
         } catch (err) {
           statsd.timing(

diff --git a/packages/fxa-auth-server/test/local/pushbox.js b/packages/fxa-auth-server/test/local/pushbox.js
@@ -97,8 +97,7 @@ describe('pushbox', () => {
           );
           sinon.assert.calledOnceWithExactly(
             mockStatsD.increment,
-            'pushbox.db.store',
-            { uid: mockUid, deviceId: mockDeviceIds[0] }
+            'pushbox.db.store'
           );
           assert.equal(index, '12');
         });