Tweak production proxy setup (#129)

* Tweak production proxy setup

* Run proxy.js in watch mode

* Update test api port

Author: nojaf

README.md

@@ -264,6 +264,9 @@ Minor differences:
 
 - Resource attributes that come from the server are prefixed with `resource.`. Since we have all attributes available, we need a way to tell which ones belong to the resource and which belong to the span. Tempo is a bit odd: if you request `select(resource.serviceAttributeName)`, it will appear on the span as `serviceAttributeName`, but because you requested it in the select you can trace it back to a resource attribute. In production, however, we receive all values without requesting them, so the `resource.` prefix is necessary to identify which attributes are resource attributes.
 
+PS: The production API needs to run on the `0.0.0.0` interface before `http://host.docker.internal:5259/tempo` can work in the `bun run server:local` setup.
+Check if "applicationUrl" in `launchSettings.json` has this.
+
 ## API discrepancies
 
 To differentiate between the Grafana Tempo API and the G-Research–flavoured Tempo API, the plugin uses a panel setting called "Enable G-Research Tempo API support". When enabled, the plugin will use the G-Research custom API features like child count support. When disabled, it works with the standard Grafana Tempo API.

local-tempo-docker-compose.yml

@@ -12,7 +12,7 @@ services:
     volumes:
       - ./scripts/proxy.js:/app/proxy.js
     working_dir: /app
-    command: bun run proxy.js
+    command: bun run --watch proxy.js
     ports:
       - '3200:3200'
     extra_hosts:

scripts/proxy.js

@@ -1,10 +1,11 @@
 Bun.serve({
   port: 3200,
   hostname: '0.0.0.0',
+  idleTimeout: 255,
   async fetch(req) {
     const url = new URL(req.url);
     const targetPath = url.pathname;
-    let targetUrl = `http://host.docker.internal:5359/tempo${targetPath}`;
+    let targetUrl = `http://host.docker.internal:5259/tempo${targetPath}`;
     if (url.searchParams.size > 0) {
       targetUrl += `?${url.searchParams.toString()}`;
     }

tests/test-api.ts

@@ -15,7 +15,7 @@ function filterSpansInPlace(filter, response: SearchResponse) {
 }
 
 Bun.serve({
-  port: 5359,
+  port: 5259,
   hostname: '0.0.0.0',
   routes: {
     '/tempo/api/echo': new Response('echo', { headers: { 'Content-Type': 'text/plain' } }),