Commit f1aedd9
Remove url.full from default OpenTelemetry metrics attributes (#2815)
## Summary
- Remove `url.full` from default OpenTelemetry metrics request
attributes to avoid high-cardinality metric explosion
- Users who need the full URL can opt in via
`OpenTelemetryDefaults.requestAttributesWithFullUrl`
- Add tests verifying `url.full` is absent by default and present when
explicitly opted in
## Test plan
- [ ] Verify new tests pass: default attributes exclude `url.full`,
`requestAttributesWithFullUrl` includes it
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>1 parent 831b2e7 commit f1aedd9
File tree
2 files changed
+36
-2
lines changed- observability/opentelemetry-backend/src
- main/scala/sttp/client4/opentelemetry
- test/scala/sttp/client4/opentelemetry
2 files changed
+36
-2
lines changedLines changed: 0 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
6 | | - | |
7 | 6 | | |
8 | 7 | | |
9 | 8 | | |
| |||
28 | 27 | | |
29 | 28 | | |
30 | 29 | | |
31 | | - | |
32 | 30 | | |
33 | 31 | | |
34 | 32 | | |
| |||
Lines changed: 36 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
276 | 276 | | |
277 | 277 | | |
278 | 278 | | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
279 | 315 | | |
280 | 316 | | |
281 | 317 | | |
| |||
0 commit comments