make view-validation failures warnings, not errors

server/routes/pixel.ts

@@ -3,8 +3,8 @@ import jwt from 'jsonwebtoken';
 
 import { TRACKER_JWT_SECRET } from '../config';
 import { TrackerTokenPayload } from '../types';
-import reportError from '../utils/report-error';
-import logError from '../utils/log-error';
+import reportMessage from '../utils/report-message';
+import logMessage from '../utils/log-message';
 import View from '../models/view';
 import noCacheMiddleware from '../middleware/no-cache';
 
@@ -46,8 +46,8 @@ router.get('/pixel.gif', async (req, res) => {
       `Logged view | Canonical: ${view.canonical} | Domain: ${view.domain} | Source: ${view.source}`
     );
   } catch (error) {
-    logError(error);
-    reportError(error);
+    logMessage(error.message);
+    reportMessage({ message: error.message });
   } finally {
     res
       .set({

server/utils/log-message.ts

@@ -0,0 +1,4 @@
+export default function logMessage(message: string): void {
+  // eslint-disable-next-line no-console
+  console.error(message);
+}

server/utils/report-error.ts

@@ -1,7 +1,5 @@
-import { captureException, withScope } from '@sentry/node';
+import { captureException } from '@sentry/node';
 
 export default function reportError(error: Error): void {
-  withScope(() => {
-    captureException(error);
-  });
+  captureException(error);
 }

server/utils/report-message.ts

@@ -7,8 +7,5 @@ export default function reportMessage({
   message: string;
   severity?: Severity;
 }): void {
-  // eslint-disable-next-line no-console
-  console.error(message);
-
   captureMessage(message, severity);
 }