fix(diagnostics): drop snake case otel ids (#72645) · openclaw/openclaw@e0bafc5

Recent Commits to openclaw:main

Lion0710 · 2026-05-23 · via Recent Commits to openclaw:main

File tree

extensions/diagnostics-otel/src

Original file line number	Diff line number	Diff line change
`@@ -97,6 +97,7 @@ Docs: https://docs.openclaw.ai`
`97`	`97`	`- Control UI/logs: strip ANSI escape sequences from displayed Gateway log messages so color codes no longer appear as raw text. Fixes #64399. Thanks @guguangxin-eng.`
`98`	`98`	- Docker: pre-create the workspace and auth-profile config mount points with `node` ownership so first-run named volumes do not start root-owned. Fixes #85076. Thanks @Noerr.
`99`	`99`	`- Telegram: pass configured markdown table mode through outbound markdown chunking so chunked sends render tables consistently. Fixes #85085. Thanks @ShuaiHui.`
	`100`	`+- Diagnostics/OTel: drop snake_case diagnostic id attributes alongside camelCase ids so exported telemetry cannot leak run, session, message, chat, trace, or tool-call identifiers. (#72645) Thanks @Lion0710.`
`100`	`101`	`- CLI/update: preserve managed Gateway service environment during package cutovers so macOS LaunchAgent repair/restart reads the pre-update service state instead of caller shell state. (#83026)`
`101`	`102`	- Agents/providers: honor per-model `api` and `baseUrl` overrides in custom provider auth hooks and transport selection. Fixes #80487. (#80488) Thanks @huveewomg.
`102`	`103`	`- Gateway/restart: eager-load the lifecycle runtime before in-place upgrade signal handling so package replacement does not deadlock restart imports. (#84890) Thanks @myps6415.`

Original file line number	Diff line number	Diff line change
`@@ -395,6 +395,62 @@ describe("diagnostics-otel service", () => {`
`395`	`395`	`}`
`396`	`396`	`});`
`397`	`397`
	`398`	`+test("drops camelCase and snake_case diagnostic id log attributes before export", async () => {`
	`399`	`+const emitCall = await emitAndCaptureLog({`
	`400`	`+level: "INFO",`
	`401`	`+message: "diagnostic id attributes",`
	`402`	`+attributes: {`
	`403`	`+callId: "call-camel",`
	`404`	`+call_id: "call-snake",`
	`405`	`+chatId: "chat-camel",`
	`406`	`+chat_id: "chat-snake",`
	`407`	`+messageId: "message-camel",`
	`408`	`+message_id: "message-snake",`
	`409`	`+parentSpanId: "parent-camel",`
	`410`	`+parent_span_id: "parent-snake",`
	`411`	`+runId: "run-camel",`
	`412`	`+run_id: "run-snake",`
	`413`	`+sessionId: "session-camel",`
	`414`	`+session_id: "session-snake",`
	`415`	`+sessionKey: "session-key-camel",`
	`416`	`+session_key: "session-key-snake",`
	`417`	`+spanId: "span-camel",`
	`418`	`+span_id: "span-snake",`
	`419`	`+toolCallId: "tool-camel",`
	`420`	`+tool_call_id: "tool-snake",`
	`421`	`+traceId: "trace-camel",`
	`422`	`+trace_id: "trace-snake",`
	`423`	`+provider: "openai",`
	`424`	`+},`
	`425`	`+});`
	`426`	`+`
	`427`	`+expect(emitCall.attributes?.["openclaw.provider"]).toBe("openai");`
	`428`	`+for (const key of [`
	`429`	`+"openclaw.callId",`
	`430`	`+"openclaw.call_id",`
	`431`	`+"openclaw.chatId",`
	`432`	`+"openclaw.chat_id",`
	`433`	`+"openclaw.messageId",`
	`434`	`+"openclaw.message_id",`
	`435`	`+"openclaw.parentSpanId",`
	`436`	`+"openclaw.parent_span_id",`
	`437`	`+"openclaw.runId",`
	`438`	`+"openclaw.run_id",`
	`439`	`+"openclaw.sessionId",`
	`440`	`+"openclaw.session_id",`
	`441`	`+"openclaw.sessionKey",`
	`442`	`+"openclaw.session_key",`
	`443`	`+"openclaw.spanId",`
	`444`	`+"openclaw.span_id",`
	`445`	`+"openclaw.toolCallId",`
	`446`	`+"openclaw.tool_call_id",`
	`447`	`+"openclaw.traceId",`
	`448`	`+"openclaw.trace_id",`
	`449`	`+]) {`
	`450`	`+expect(Object.hasOwn(emitCall.attributes ?? {}, key)).toBe(false);`
	`451`	`+}`
	`452`	`+});`
	`453`	`+`
`398`	`454`	`test("records message-flow metrics and spans", async () => {`
`399`	`455`	`const service = createDiagnosticsOtelService();`
`400`	`456`	`const ctx = createOtelContext(OTEL_TEST_ENDPOINT, { traces: true, metrics: true, logs: true });`

Original file line number	Diff line number	Diff line change
`@@ -32,15 +32,25 @@ import {`
`32`	`32`	`const DEFAULT_SERVICE_NAME = "openclaw";`
`33`	`33`	`const DROPPED_OTEL_ATTRIBUTE_KEYS = new Set([`
`34`	`34`	`"openclaw.callId",`
	`35`	`+"openclaw.call_id",`
`35`	`36`	`"openclaw.chatId",`
	`37`	`+"openclaw.chat_id",`
`36`	`38`	`"openclaw.messageId",`
	`39`	`+"openclaw.message_id",`
`37`	`40`	`"openclaw.parentSpanId",`
	`41`	`+"openclaw.parent_span_id",`
`38`	`42`	`"openclaw.runId",`
	`43`	`+"openclaw.run_id",`
`39`	`44`	`"openclaw.sessionId",`
	`45`	`+"openclaw.session_id",`
`40`	`46`	`"openclaw.sessionKey",`
	`47`	`+"openclaw.session_key",`
`41`	`48`	`"openclaw.spanId",`
	`49`	`+"openclaw.span_id",`
`42`	`50`	`"openclaw.toolCallId",`
	`51`	`+"openclaw.tool_call_id",`
`43`	`52`	`"openclaw.traceId",`
	`53`	`+"openclaw.trace_id",`
`44`	`54`	`]);`
`45`	`55`	`const LOW_CARDINALITY_VALUE_RE = /^[A-Za-z0-9_.:-]{1,120}$/u;`
`46`	`56`	`const MAX_OTEL_CONTENT_ATTRIBUTE_CHARS = 4 * 1024;`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main

File tree