[dashboard] improve metrics reporting (#18755)

* [dashboard] add session id to error reporting

* [public-api] fix conversion to seconds

* [public-api] don't try server-stream test if not authenticated

Author: akosyakov

components/dashboard/resolver.js

@@ -3,10 +3,23 @@
  * Licensed under the GNU Affero General Public License (AGPL).
  * See License.AGPL.txt in the project root for license information.
  */
+
+/*
+ * Resolver allows to reconstruct stack traces from obfuscated stack traces for the dashboard.
+ * Usage:
+ *  node resolver.js < obfuscated-stack-trace.txt
+ *
+ * OR
+ *
+ *  node resolver.js <<EOF
+ *  obfuscated stack trace
+ *  EOF
+ */
+
 //@ts-check
-const path = require('path');
-const fetch = require('node-fetch').default;
-const { SourceMapConsumer } = require('source-map');
+const path = require("path");
+const fetch = require("node-fetch").default;
+const { SourceMapConsumer } = require("source-map");
 
 const sourceMapCache = {};
 
@@ -27,10 +40,10 @@ async function fetchSourceMap(jsUrl) {
     // Extract source map URL from the JS file
     const mapUrlMatch = jsContent.match(/\/\/#\s*sourceMappingURL=(.+)/);
     if (!mapUrlMatch) {
-        throw new Error('Source map URL not found');
+        throw new Error("Source map URL not found");
     }
 
-    const mapUrl = new URL(mapUrlMatch[1], jsUrl).href;  // Resolve relative URL
+    const mapUrl = new URL(mapUrlMatch[1], jsUrl).href; // Resolve relative URL
     const mapResponse = await fetch(mapUrl);
     const mapData = await mapResponse.json();
 
@@ -40,7 +53,7 @@ async function fetchSourceMap(jsUrl) {
     return mapData;
 }
 
-const BASE_PATH = '/workspace/gitpod/components';
+const BASE_PATH = "/workspace/gitpod/components";
 
 async function resolveLine(line) {
     const jsUrl = extractJsUrlFromLine(line);
@@ -53,7 +66,7 @@ async function resolveLine(line) {
         return line;
     }
 
-    const functionName = matches[1] || '';
+    const functionName = matches[1] || "";
     const lineNum = Number(matches[3]);
     const colNum = Number(matches[4]);
 
@@ -69,18 +82,17 @@ async function resolveLine(line) {
     return line;
 }
 
+let obfuscatedTrace = "";
 
-let obfuscatedTrace = '';
-
-process.stdin.on('data', function(data) {
+process.stdin.on("data", function (data) {
     obfuscatedTrace += data;
 });
 
-process.stdin.on('end', async function() {
-    const lines = obfuscatedTrace.split('\n');
+process.stdin.on("end", async function () {
+    const lines = obfuscatedTrace.split("\n");
     const resolvedLines = await Promise.all(lines.map(resolveLine));
-    const resolvedTrace = resolvedLines.join('\n');
-    console.log('\nResolved Stack Trace:\n');
+    const resolvedTrace = resolvedLines.join("\n");
+    console.log("\nResolved Stack Trace:\n");
     console.log(resolvedTrace);
 });
 

components/dashboard/src/service/metrics.ts

@@ -7,6 +7,7 @@
 import { GitpodHostUrl } from "@gitpod/gitpod-protocol/lib/util/gitpod-host-url";
 import { MetricsReporter } from "@gitpod/public-api/lib/metrics";
 import { getExperimentsClient } from "../experiments/client";
+import { v4 } from "uuid";
 
 const originalConsoleError = console.error;
 
@@ -40,7 +41,9 @@ console.error = function (...args) {
     reportError(...args);
 };
 
-let commonDetails = {};
+const commonDetails = {
+    sessionId: v4(),
+};
 export function updateCommonErrorDetails(update: { [key: string]: string | undefined }) {
     Object.assign(commonDetails, update);
 }

components/dashboard/src/service/service.tsx

@@ -116,10 +116,12 @@ function testPublicAPI(service: any): void {
 
         // emulates server side streaming with public API
         while (true) {
-            const isTest = await getExperimentsClient().getValueAsync("public_api_dummy_reliability_test", false, {
-                user,
-                gitpodHost: window.location.host,
-            });
+            const isTest =
+                !!user &&
+                (await getExperimentsClient().getValueAsync("public_api_dummy_reliability_test", false, {
+                    user,
+                    gitpodHost: window.location.host,
+                }));
             if (isTest) {
                 try {
                     let previousCount = 0;

components/public-api/typescript/src/metrics.ts

@@ -131,7 +131,7 @@ class PrometheusClientCallMetrics {
             const start = performance.now();
             return (endLabels) => {
                 const delta = performance.now() - start;
-                const value = delta / 1e9;
+                const value = delta / 1000;
                 this.handledSecondsHistogram.labels(Object.assign(startLabels, endLabels)).observe(value);
                 return value;
             };