feat(node): Add server.address to nodejs logs (#16006)

`server.address` is the equivalent to `server_name` we attach to errors
and transactions. See the spec for the attribute here:
https://getsentry.github.io/sentry-conventions/generated/attributes/server.html#serveraddress

resolves https://linear.app/getsentry/issue/LOGS-32

Author: AbhiPrasad

packages/node/src/sdk/client.ts

@@ -21,11 +21,12 @@ export class NodeClient extends ServerRuntimeClient<NodeClientOptions> {
   private _logOnExitFlushListener: (() => void) | undefined;
 
   public constructor(options: NodeClientOptions) {
+    const serverName = options.serverName || global.process.env.SENTRY_NAME || os.hostname();
     const clientOptions: ServerRuntimeClientOptions = {
       ...options,
       platform: 'node',
       runtime: { name: 'node', version: global.process.version },
-      serverName: options.serverName || global.process.env.SENTRY_NAME || os.hostname(),
+      serverName,
     };
 
     if (options.openTelemetryInstrumentations) {
@@ -47,6 +48,15 @@ export class NodeClient extends ServerRuntimeClient<NodeClientOptions> {
         _INTERNAL_flushLogsBuffer(this);
       };
 
+      if (serverName) {
+        this.on('beforeCaptureLog', log => {
+          log.attributes = {
+            ...log.attributes,
+            'server.address': serverName,
+          };
+        });
+      }
+
       process.on('beforeExit', this._logOnExitFlushListener);
     }
   }

packages/node/test/sdk/client.test.ts

@@ -1,7 +1,7 @@
 import * as os from 'os';
 import { ProxyTracer } from '@opentelemetry/api';
 import * as opentelemetryInstrumentationPackage from '@opentelemetry/instrumentation';
-import type { Event, EventHint } from '@sentry/core';
+import type { Event, EventHint, Log } from '@sentry/core';
 import { SDK_VERSION, Scope, getCurrentScope, getGlobalScope, getIsolationScope } from '@sentry/core';
 import { setOpenTelemetryContextAsyncContextStrategy } from '@sentry/opentelemetry';
 import { afterEach, beforeEach, describe, expect, it, test, vi } from 'vitest';
@@ -281,4 +281,32 @@ describe('NodeClient', () => {
       }),
     );
   });
+
+  describe('log capture', () => {
+    it('adds server name to log attributes', () => {
+      const options = getDefaultNodeClientOptions({ _experiments: { enableLogs: true } });
+      const client = new NodeClient(options);
+
+      const log: Log = { level: 'info', message: 'test message', attributes: {} };
+      client.emit('beforeCaptureLog', log);
+
+      expect(log.attributes).toEqual({
+        'server.address': expect.any(String),
+      });
+    });
+
+    it('preserves existing log attributes', () => {
+      const serverName = 'test-server';
+      const options = getDefaultNodeClientOptions({ serverName, _experiments: { enableLogs: true } });
+      const client = new NodeClient(options);
+
+      const log: Log = { level: 'info', message: 'test message', attributes: { 'existing.attr': 'value' } };
+      client.emit('beforeCaptureLog', log);
+
+      expect(log.attributes).toEqual({
+        'existing.attr': 'value',
+        'server.address': serverName,
+      });
+    });
+  });
 });