Change hash computation for protobuf to better represent impacting changes + save proto number in schema #8201

vandonr · 2025-01-14T16:35:59Z

What Does This Do

Change the way the hash is computed for protobuf schemas:

to catch some changes that impact the wire format but were not changing the hash before (like changing a field number for instance)
to stay constant on changes that don't impact the wire format, but were changing the hash before (like reorganizing fields without changing their number)
It should also be marginally faster as we don't hash a whole string, and also isolates the hash from json shennanigans like the OpenAPI version for instance.

This new hash computation is aligned with what's done in the dotnet version of the instrumentation in DataDog/dd-trace-dotnet#6166

ℹ️ Also add protobuf number to the OpenAPI schema saved, so that we can reconstruct the hash if needed, and also because it's an important component of a protobuf schema.

Additional Notes

Depending on how schema hashes are used now in DSM, when deploying this, it may look to users as if all their schemas changed

Contributor Checklist

Format the title according the contribution guidelines
Assign the type: and (comp: or inst:) labels in addition to any usefull labels
Don't use close, fix or any linking keywords when referencing an issue.
Use solves instead, and assign the PR milestone to the issue
Update the public documentation in case of new configuration flag or behavior

Jira ticket: [PROJ-IDENT]

github-actions · 2025-01-14T16:36:15Z

Hi! 👋 Thanks for your pull request! 🎉

To help us review it, please make sure to:

Add at least one type, and one component or instrumentation label to the pull request

If you need help, please check our contributing guidelines.

pr-commenter · 2025-01-14T17:11:13Z

Benchmarks

Startup

Parameters

	Baseline	Candidate
baseline_or_candidate	baseline	candidate
git_branch	master	vandonr/proto
git_commit_date	1736890492	1736938770
git_commit_sha	`8b63e5a`	`d702a13`
release_version	1.46.0-SNAPSHOT~8b63e5a85b	1.46.0-SNAPSHOT~d702a13e68

See matching parameters

	Baseline	Candidate
application	insecure-bank	insecure-bank
ci_job_date	1736941339	1736941339
ci_job_id	767038699	767038699
ci_pipeline_id	53054901	53054901
cpu_model	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
module	Agent	Agent
parent	None	None
variant	iast	iast

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 58 metrics, 5 unstable metrics.

Startup time reports for insecure-bank

gantt
    title insecure-bank - global startup overhead: candidate=1.46.0-SNAPSHOT~d702a13e68, baseline=1.46.0-SNAPSHOT~8b63e5a85b

    dateFormat X
    axisFormat %s
section tracing
Agent [baseline] (1.068 s) : 0, 1068375
Total [baseline] (8.666 s) : 0, 8665547
Agent [candidate] (1.07 s) : 0, 1069896
Total [candidate] (8.677 s) : 0, 8677345
section iast
Agent [baseline] (1.186 s) : 0, 1186240
Total [baseline] (9.243 s) : 0, 9242833
Agent [candidate] (1.185 s) : 0, 1184695
Total [candidate] (9.232 s) : 0, 9231735
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.183 s) : 0, 1183065
Total [baseline] (9.186 s) : 0, 9186051
Agent [candidate] (1.198 s) : 0, 1197808
Total [candidate] (9.259 s) : 0, 9259134
section iast_TELEMETRY_OFF
Agent [baseline] (1.185 s) : 0, 1185249
Total [baseline] (9.257 s) : 0, 9256627
Agent [candidate] (1.177 s) : 0, 1177083
Total [candidate] (9.261 s) : 0, 9261177

baseline results

Module	Variant	Duration	Δ tracing
Agent	tracing	1.068 s	-
Agent	iast	1.186 s	117.865 ms (11.0%)
Agent	iast_HARDCODED_SECRET_DISABLED	1.183 s	114.689 ms (10.7%)
Agent	iast_TELEMETRY_OFF	1.185 s	116.874 ms (10.9%)
Total	tracing	8.666 s	-
Total	iast	9.243 s	577.286 ms (6.7%)
Total	iast_HARDCODED_SECRET_DISABLED	9.186 s	520.504 ms (6.0%)
Total	iast_TELEMETRY_OFF	9.257 s	591.08 ms (6.8%)

candidate results

Module	Variant	Duration	Δ tracing
Agent	tracing	1.07 s	-
Agent	iast	1.185 s	114.799 ms (10.7%)
Agent	iast_HARDCODED_SECRET_DISABLED	1.198 s	127.912 ms (12.0%)
Agent	iast_TELEMETRY_OFF	1.177 s	107.187 ms (10.0%)
Total	tracing	8.677 s	-
Total	iast	9.232 s	554.39 ms (6.4%)
Total	iast_HARDCODED_SECRET_DISABLED	9.259 s	581.789 ms (6.7%)
Total	iast_TELEMETRY_OFF	9.261 s	583.832 ms (6.7%)

gantt
    title insecure-bank - break down per module: candidate=1.46.0-SNAPSHOT~d702a13e68, baseline=1.46.0-SNAPSHOT~8b63e5a85b

    dateFormat X
    axisFormat %s
section tracing
BytebuddyAgent [baseline] (723.73 ms) : 0, 723730
BytebuddyAgent [candidate] (724.361 ms) : 0, 724361
GlobalTracer [baseline] (259.295 ms) : 0, 259295
GlobalTracer [candidate] (259.745 ms) : 0, 259745
AppSec [baseline] (55.975 ms) : 0, 55975
AppSec [candidate] (56.713 ms) : 0, 56713
Remote Config [baseline] (748.722 µs) : 0, 749
Remote Config [candidate] (739.772 µs) : 0, 740
Telemetry [baseline] (13.386 ms) : 0, 13386
Telemetry [candidate] (13.106 ms) : 0, 13106
section iast
BytebuddyAgent [baseline] (836.016 ms) : 0, 836016
BytebuddyAgent [candidate] (833.473 ms) : 0, 833473
GlobalTracer [baseline] (245.854 ms) : 0, 245854
GlobalTracer [candidate] (247.25 ms) : 0, 247250
AppSec [baseline] (58.134 ms) : 0, 58134
AppSec [candidate] (58.104 ms) : 0, 58104
IAST [baseline] (21.698 ms) : 0, 21698
IAST [candidate] (21.365 ms) : 0, 21365
Remote Config [baseline] (677.889 µs) : 0, 678
Remote Config [candidate] (670.389 µs) : 0, 670
Telemetry [baseline] (8.848 ms) : 0, 8848
Telemetry [candidate] (8.774 ms) : 0, 8774
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (831.716 ms) : 0, 831716
BytebuddyAgent [candidate] (842.058 ms) : 0, 842058
GlobalTracer [baseline] (247.048 ms) : 0, 247048
GlobalTracer [candidate] (250.044 ms) : 0, 250044
AppSec [baseline] (58.295 ms) : 0, 58295
AppSec [candidate] (58.868 ms) : 0, 58868
IAST [baseline] (21.494 ms) : 0, 21494
IAST [candidate] (21.989 ms) : 0, 21989
Remote Config [baseline] (672.115 µs) : 0, 672
Remote Config [candidate] (695.792 µs) : 0, 696
Telemetry [baseline] (8.85 ms) : 0, 8850
Telemetry [candidate] (9.026 ms) : 0, 9026
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (834.381 ms) : 0, 834381
BytebuddyAgent [candidate] (827.202 ms) : 0, 827202
GlobalTracer [baseline] (248.062 ms) : 0, 248062
GlobalTracer [candidate] (246.671 ms) : 0, 246671
AppSec [baseline] (57.816 ms) : 0, 57816
AppSec [candidate] (57.932 ms) : 0, 57932
IAST [baseline] (20.692 ms) : 0, 20692
IAST [candidate] (20.917 ms) : 0, 20917
Remote Config [baseline] (668.196 µs) : 0, 668
Remote Config [candidate] (679.741 µs) : 0, 680
Telemetry [baseline] (8.584 ms) : 0, 8584
Telemetry [candidate] (8.696 ms) : 0, 8696

Startup time reports for petclinic

gantt
    title petclinic - global startup overhead: candidate=1.46.0-SNAPSHOT~d702a13e68, baseline=1.46.0-SNAPSHOT~8b63e5a85b

    dateFormat X
    axisFormat %s
section tracing
Agent [baseline] (1.068 s) : 0, 1067857
Total [baseline] (10.626 s) : 0, 10625789
Agent [candidate] (1.055 s) : 0, 1054605
Total [candidate] (10.435 s) : 0, 10435009
section appsec
Agent [baseline] (1.187 s) : 0, 1186807
Total [baseline] (10.731 s) : 0, 10731266
Agent [candidate] (1.19 s) : 0, 1190416
Total [candidate] (10.718 s) : 0, 10717949
section iast
Agent [baseline] (1.197 s) : 0, 1196661
Total [baseline] (11.003 s) : 0, 11003404
Agent [candidate] (1.191 s) : 0, 1190559
Total [candidate] (11.014 s) : 0, 11014071
section profiling
Agent [baseline] (1.255 s) : 0, 1255476
Total [baseline] (10.832 s) : 0, 10832349
Agent [candidate] (1.259 s) : 0, 1258748
Total [candidate] (10.82 s) : 0, 10820480

baseline results

Module	Variant	Duration	Δ tracing
Agent	tracing	1.068 s	-
Agent	appsec	1.187 s	118.95 ms (11.1%)
Agent	iast	1.197 s	128.804 ms (12.1%)
Agent	profiling	1.255 s	187.619 ms (17.6%)
Total	tracing	10.626 s	-
Total	appsec	10.731 s	105.476 ms (1.0%)
Total	iast	11.003 s	377.615 ms (3.6%)
Total	profiling	10.832 s	206.56 ms (1.9%)

candidate results

Module	Variant	Duration	Δ tracing
Agent	tracing	1.055 s	-
Agent	appsec	1.19 s	135.811 ms (12.9%)
Agent	iast	1.191 s	135.954 ms (12.9%)
Agent	profiling	1.259 s	204.143 ms (19.4%)
Total	tracing	10.435 s	-
Total	appsec	10.718 s	282.94 ms (2.7%)
Total	iast	11.014 s	579.063 ms (5.5%)
Total	profiling	10.82 s	385.471 ms (3.7%)

gantt
    title petclinic - break down per module: candidate=1.46.0-SNAPSHOT~d702a13e68, baseline=1.46.0-SNAPSHOT~8b63e5a85b

    dateFormat X
    axisFormat %s
section tracing
BytebuddyAgent [baseline] (723.168 ms) : 0, 723168
BytebuddyAgent [candidate] (713.553 ms) : 0, 713553
GlobalTracer [baseline] (257.995 ms) : 0, 257995
GlobalTracer [candidate] (255.844 ms) : 0, 255844
AppSec [baseline] (55.68 ms) : 0, 55680
AppSec [candidate] (55.232 ms) : 0, 55232
Remote Config [baseline] (728.93 µs) : 0, 729
Remote Config [candidate] (725.709 µs) : 0, 726
Telemetry [baseline] (15.137 ms) : 0, 15137
Telemetry [candidate] (14.224 ms) : 0, 14224
section appsec
BytebuddyAgent [baseline] (729.988 ms) : 0, 729988
BytebuddyAgent [candidate] (732.122 ms) : 0, 732122
GlobalTracer [baseline] (252.704 ms) : 0, 252704
GlobalTracer [candidate] (253.387 ms) : 0, 253387
AppSec [baseline] (170.298 ms) : 0, 170298
AppSec [candidate] (171.057 ms) : 0, 171057
IAST [baseline] (19.349 ms) : 0, 19349
IAST [candidate] (19.614 ms) : 0, 19614
Remote Config [baseline] (669.009 µs) : 0, 669
Remote Config [candidate] (672.595 µs) : 0, 673
Telemetry [baseline] (8.567 ms) : 0, 8567
Telemetry [candidate] (8.283 ms) : 0, 8283
section iast
BytebuddyAgent [baseline] (843.349 ms) : 0, 843349
BytebuddyAgent [candidate] (837.701 ms) : 0, 837701
GlobalTracer [baseline] (248.64 ms) : 0, 248640
GlobalTracer [candidate] (248.162 ms) : 0, 248162
AppSec [baseline] (58.226 ms) : 0, 58226
AppSec [candidate] (58.294 ms) : 0, 58294
IAST [baseline] (21.655 ms) : 0, 21655
IAST [candidate] (21.49 ms) : 0, 21490
Remote Config [baseline] (671.015 µs) : 0, 671
Remote Config [candidate] (676.398 µs) : 0, 676
Telemetry [baseline] (8.877 ms) : 0, 8877
Telemetry [candidate] (8.886 ms) : 0, 8886
section profiling
BytebuddyAgent [baseline] (703.186 ms) : 0, 703186
BytebuddyAgent [candidate] (705.556 ms) : 0, 705556
GlobalTracer [baseline] (349.931 ms) : 0, 349931
GlobalTracer [candidate] (351.677 ms) : 0, 351677
AppSec [baseline] (54.755 ms) : 0, 54755
AppSec [candidate] (53.99 ms) : 0, 53990
Remote Config [baseline] (658.881 µs) : 0, 659
Remote Config [candidate] (663.146 µs) : 0, 663
Telemetry [baseline] (8.88 ms) : 0, 8880
Telemetry [candidate] (8.869 ms) : 0, 8869
ProfilingAgent [baseline] (95.973 ms) : 0, 95973
ProfilingAgent [candidate] (95.969 ms) : 0, 95969
Profiling [baseline] (95.998 ms) : 0, 95998
Profiling [candidate] (95.994 ms) : 0, 95994

Load

Parameters

	Baseline	Candidate
baseline_or_candidate	baseline	candidate
end_time	2025-01-15T11:14:03	2025-01-15T11:21:04
git_branch	master	vandonr/proto
git_commit_date	1736890492	1736938770
git_commit_sha	`8b63e5a`	`d702a13`
release_version	1.46.0-SNAPSHOT~8b63e5a85b	1.46.0-SNAPSHOT~d702a13e68
start_time	2025-01-15T11:13:49	2025-01-15T11:20:51

See matching parameters

	Baseline	Candidate
application	insecure-bank	insecure-bank
ci_job_date	1736940417	1736940417
ci_job_id	767038700	767038700
ci_pipeline_id	53054901	53054901
cpu_model	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
variant	iast	iast

Summary

Found 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 16 unstable metrics.

scenario	Δ mean http_req_duration	Δ mean throughput	candidate mean http_req_duration	candidate mean throughput	baseline mean http_req_duration	baseline mean throughput
scenario:load:petclinic:profiling	better [-94.857µs; -43.452µs] or [-6.059%; -2.776%]	unstable [-534.663op/s; +534.663op/s] or [-17.377%; +17.377%]	1.496ms	3076.923op/s	1.566ms	3076.923op/s

Request duration reports for insecure-bank

gantt
    title insecure-bank - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~d702a13e68, baseline=1.46.0-SNAPSHOT~8b63e5a85b
    dateFormat X
    axisFormat %s
section baseline
no_agent (382.414 µs) : 361, 404
.   : milestone, 382,
iast (499.102 µs) : 477, 521
.   : milestone, 499,
iast_FULL (662.328 µs) : 641, 684
.   : milestone, 662,
iast_GLOBAL (517.885 µs) : 496, 539
.   : milestone, 518,
iast_HARDCODED_SECRET_DISABLED (502.426 µs) : 481, 524
.   : milestone, 502,
iast_INACTIVE (453.792 µs) : 433, 475
.   : milestone, 454,
iast_TELEMETRY_OFF (484.824 µs) : 463, 506
.   : milestone, 485,
tracing (453.682 µs) : 433, 475
.   : milestone, 454,
section candidate
no_agent (379.713 µs) : 360, 400
.   : milestone, 380,
iast (497.694 µs) : 476, 519
.   : milestone, 498,
iast_FULL (655.235 µs) : 634, 677
.   : milestone, 655,
iast_GLOBAL (528.697 µs) : 506, 552
.   : milestone, 529,
iast_HARDCODED_SECRET_DISABLED (494.784 µs) : 473, 516
.   : milestone, 495,
iast_INACTIVE (453.812 µs) : 432, 475
.   : milestone, 454,
iast_TELEMETRY_OFF (485.49 µs) : 464, 507
.   : milestone, 485,
tracing (457.438 µs) : 437, 478
.   : milestone, 457,

baseline results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	382.414 µs [360.743 µs, 404.086 µs]	-
iast	499.102 µs [477.485 µs, 520.719 µs]	116.688 µs (30.5%)
iast_FULL	662.328 µs [640.543 µs, 684.113 µs]	279.914 µs (73.2%)
iast_GLOBAL	517.885 µs [496.497 µs, 539.273 µs]	135.471 µs (35.4%)
iast_HARDCODED_SECRET_DISABLED	502.426 µs [480.69 µs, 524.162 µs]	120.012 µs (31.4%)
iast_INACTIVE	453.792 µs [432.871 µs, 474.712 µs]	71.377 µs (18.7%)
iast_TELEMETRY_OFF	484.824 µs [463.368 µs, 506.279 µs]	102.41 µs (26.8%)
tracing	453.682 µs [432.808 µs, 474.556 µs]	71.268 µs (18.6%)

candidate results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	379.713 µs [359.798 µs, 399.627 µs]	-
iast	497.694 µs [476.05 µs, 519.338 µs]	117.981 µs (31.1%)
iast_FULL	655.235 µs [633.713 µs, 676.757 µs]	275.522 µs (72.6%)
iast_GLOBAL	528.697 µs [505.808 µs, 551.585 µs]	148.984 µs (39.2%)
iast_HARDCODED_SECRET_DISABLED	494.784 µs [473.082 µs, 516.485 µs]	115.071 µs (30.3%)
iast_INACTIVE	453.812 µs [432.447 µs, 475.177 µs]	74.1 µs (19.5%)
iast_TELEMETRY_OFF	485.49 µs [464.069 µs, 506.912 µs]	105.778 µs (27.9%)
tracing	457.438 µs [436.542 µs, 478.334 µs]	77.725 µs (20.5%)

Request duration reports for petclinic

gantt
    title petclinic - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~d702a13e68, baseline=1.46.0-SNAPSHOT~8b63e5a85b
    dateFormat X
    axisFormat %s
section baseline
no_agent (1.343 ms) : 1323, 1364
.   : milestone, 1343,
appsec (1.764 ms) : 1740, 1787
.   : milestone, 1764,
appsec_no_iast (1.756 ms) : 1731, 1782
.   : milestone, 1756,
iast (1.499 ms) : 1476, 1522
.   : milestone, 1499,
profiling (1.566 ms) : 1541, 1590
.   : milestone, 1566,
tracing (1.475 ms) : 1450, 1500
.   : milestone, 1475,
section candidate
no_agent (1.328 ms) : 1309, 1348
.   : milestone, 1328,
appsec (1.757 ms) : 1734, 1780
.   : milestone, 1757,
appsec_no_iast (1.76 ms) : 1737, 1783
.   : milestone, 1760,
iast (1.496 ms) : 1473, 1519
.   : milestone, 1496,
profiling (1.496 ms) : 1473, 1520
.   : milestone, 1496,
tracing (1.472 ms) : 1447, 1497
.   : milestone, 1472,

baseline results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	1.343 ms [1.323 ms, 1.364 ms]	-
appsec	1.764 ms [1.74 ms, 1.787 ms]	420.393 µs (31.3%)
appsec_no_iast	1.756 ms [1.731 ms, 1.782 ms]	412.92 µs (30.7%)
iast	1.499 ms [1.476 ms, 1.522 ms]	155.98 µs (11.6%)
profiling	1.566 ms [1.541 ms, 1.59 ms]	222.227 µs (16.5%)
tracing	1.475 ms [1.45 ms, 1.5 ms]	131.513 µs (9.8%)

candidate results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	1.328 ms [1.309 ms, 1.348 ms]	-
appsec	1.757 ms [1.734 ms, 1.78 ms]	428.709 µs (32.3%)
appsec_no_iast	1.76 ms [1.737 ms, 1.783 ms]	431.608 µs (32.5%)
iast	1.496 ms [1.473 ms, 1.519 ms]	167.631 µs (12.6%)
profiling	1.496 ms [1.473 ms, 1.52 ms]	167.887 µs (12.6%)
tracing	1.472 ms [1.447 ms, 1.497 ms]	143.304 µs (10.8%)

Dacapo

| Package | Type | Package file | Manager | Update | Change | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-datastore](https://github.com/googleapis/java-datastore) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `2.25.4` -> `2.26.0` | | [com.datadoghq:dd-trace-api](https://github.com/datadog/dd-trace-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.45.2` -> `1.46.0` | | [com.datadoghq:dd-trace-ot](https://github.com/datadog/dd-trace-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.45.2` -> `1.46.0` | | [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | | [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.30.9` -> `2.30.10` | --- ### Release Notes <details> <summary>googleapis/java-datastore (com.google.cloud:google-cloud-datastore)</summary> ### [`v2.26.0`](https://github.com/googleapis/java-datastore/blob/HEAD/CHANGELOG.md#2260-2025-01-29) ##### Features - Add firestoreInDatastoreMode for datastore emulator ([#1698](googleapis/java-datastore#1698)) ([50f106d](googleapis/java-datastore@50f106d)) ##### Dependencies - Update dependency com.google.cloud:sdk-platform-java-config to v3.42.0 ([#1725](googleapis/java-datastore#1725)) ([1cbaf22](googleapis/java-datastore@1cbaf22)) </details> <details> <summary>datadog/dd-trace-java (com.datadoghq:dd-trace-api)</summary> ### [`v1.46.0`](https://github.com/DataDog/dd-trace-java/releases/tag/v1.46.0): 1.46.0 ##### Breaking Changes > \[!WARNING] > jnr-unixsocket is now an external dependency of dd-trace-ot and must be included when deploying dd-trace-ot. > \[!NOTE] > The API `TracerScope.setAsync(boolean)`, used to manually control asynchronous span propagation, does no more apply to the scope instance but to the active span scope. ##### Components ##### Application Security Management (IAST) - 🐛 Fix String.replace instrumentation for IAST ([#8281](DataDog/dd-trace-java#8281) - [@Mariovido](https://github.com/Mariovido)) - ✨ Apply the standard nomenclature to the stacktrace configs ([#8244](DataDog/dd-trace-java#8244) - [@jandro996](https://github.com/jandro996)) - 🐛 Exclude false positive weak randomness ([#8232](DataDog/dd-trace-java#8232) - [@jandro996](https://github.com/jandro996)) - ✨ Propagation of translateEscapes of String class ([#8186](DataDog/dd-trace-java#8186) - [@sezen-datadog](https://github.com/sezen-datadog)) - ✨ Add security control metrics ([#8175](DataDog/dd-trace-java#8175) - [@jandro996](https://github.com/jandro996)) - ✨ Increase IAST propagation to StringBuffer setLength ([#8128](DataDog/dd-trace-java#8128) - [@Mariovido](https://github.com/Mariovido)) - ✨ Add IAST taint tracking for DB values ([#8072](DataDog/dd-trace-java#8072) - [@Mariovido](https://github.com/Mariovido)) ##### Application Security Management (WAF) - 🐛 Prevents a NPE when there is no subscriber for user events ([#8258](DataDog/dd-trace-java#8258) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) - ✨ Apply the standard nomenclature to the stacktrace configs ([#8244](DataDog/dd-trace-java#8244) - [@jandro996](https://github.com/jandro996)) - 🐛 Ensure cached subscriptions are cleared on reconfiguration via RC ([#8229](DataDog/dd-trace-java#8229) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) - ✨ Add support for session tracking in Vertx ([#8167](DataDog/dd-trace-java#8167) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) - ✨ Create span tag: \_dd.appsec.rasp.timeout ([#8269](DataDog/dd-trace-java#8269) - [@Mariovido](https://github.com/Mariovido)) ##### Build & Tooling - 🐛 Ensure shaded helpers have unique names when injected into class-loaders ([#8192](DataDog/dd-trace-java#8192) - [@mcculls](https://github.com/mcculls)) ##### Configuration at Runtime - 🐛 Remove filtering of `DD_SERVICE` and `DD_ENV` from the tracer ([#8176](DataDog/dd-trace-java#8176) - [@mhlidd](https://github.com/mhlidd)) ##### Continuous Integration Visibility - 🧹 Generalize TestRetryPolicy to TestExecutionPolicy ([#8302](DataDog/dd-trace-java#8302) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🧹 Parallelize CI Visibility settings requests ([#8299](DataDog/dd-trace-java#8299) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🧹 Generalize test retry logic ([#8289](DataDog/dd-trace-java#8289) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🧹 Generalize tests skipping logic ([#8288](DataDog/dd-trace-java#8288) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🧹 Remove skip and shouldBeSkipped methods from TestEventsHandler in favor of isSkippable ([#8286](DataDog/dd-trace-java#8286) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨⚡ Optimize Git repository information computation ([#8270](DataDog/dd-trace-java#8270) - [@dougqh](https://github.com/dougqh)) - ✨ Always request known tests from the backend ([#8268](DataDog/dd-trace-java#8268) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨ Fix NPE when trying to get retry analyzer in Test NG ([#8253](DataDog/dd-trace-java#8253) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🐛 Set test framework and test framework version tags atomically ([#8252](DataDog/dd-trace-java#8252) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨ Add debug logging to Android Gradle module layout logic ([#8251](DataDog/dd-trace-java#8251) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - 🐛 Fix source and destination folders computation for Android Gradle projects ([#8190](DataDog/dd-trace-java#8190) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨ Add basic Scala Weaver sbt support ([#8189](DataDog/dd-trace-java#8189) - [@daniel-mohedano](https://github.com/daniel-mohedano)) - ✨ Implement impacted tests detection ([#8188](DataDog/dd-trace-java#8188) - [@nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) ##### Data Streams Monitoring - ✨ Change hash computation for protobuf to better represent impacting changes + save proto number in schema ([#8201](DataDog/dd-trace-java#8201) - [@vandonr](https://github.com/vandonr)) ##### Database Monitoring - Add peer service tag in dbm sql commenter ([#7913](DataDog/dd-trace-java#7913) - [@jordan-wong](https://github.com/jordan-wong)) ##### Dynamic Instrumentation - ✨ Add support for SymDB to scan directories ([#8306](DataDog/dd-trace-java#8306) - [@jpbempel](https://github.com/jpbempel)) - ✨ Add SymDB report for any jar scanning failures ([#8300](DataDog/dd-trace-java#8300) - [@jpbempel](https://github.com/jpbempel)) - ✨ Use two budgets depending on type ([#8283](DataDog/dd-trace-java#8283) - [@evanchooly](https://github.com/evanchooly)) - ✨ Institute a 10 snapshot per probe per trace budget ([#8277](DataDog/dd-trace-java#8277) - [@evanchooly](https://github.com/evanchooly)) - 🐛 Avoid double snapshots for Exception Replay ([#8273](DataDog/dd-trace-java#8273) - [@jpbempel](https://github.com/jpbempel)) - ✨ Simplify code origins. Separate out snapshot generation. ([#8263](DataDog/dd-trace-java#8263) - [@evanchooly](https://github.com/evanchooly)) - ✨ Add Exception probe custom instrumentation ([#8230](DataDog/dd-trace-java#8230) - [@jpbempel](https://github.com/jpbempel)) - ✨ Enhance log probes to honor debug session tags ([#8215](DataDog/dd-trace-java#8215) - [@evanchooly](https://github.com/evanchooly)) - 🐛 Don't redact env tokens from debugger probe snapshots ([#8211](DataDog/dd-trace-java#8211) - [@watson](https://github.com/watson)) - ✨⚡ Move Trace/SpanId capture at commit time ([#8184](DataDog/dd-trace-java#8184) - [@jpbempel](https://github.com/jpbempel)) - 🐛 Capture values at entry for method probe ([#8169](DataDog/dd-trace-java#8169) - [@jpbempel](https://github.com/jpbempel)) ##### JMX fetch - 🐛 Mute JMXFetch Shutdown in progress error ([#8068](DataDog/dd-trace-java#8068) - [@ygree](https://github.com/ygree)) ##### OpenTracing - ⚠️🧹 Make jnr-unixsocket an explicit dependency of dd-trace-ot ([#8307](DataDog/dd-trace-java#8307) - [@mcculls](https://github.com/mcculls)) ##### Profiling - 🐛 Avoid unsupported API call for creating folders on windows ([#8304](DataDog/dd-trace-java#8304) - [@jbachorik](https://github.com/jbachorik)) - ✨ Tag profiles for serverless ([#8279](DataDog/dd-trace-java#8279) - [@jbachorik](https://github.com/jbachorik)) - ✨ add queue type and length to queue events ([#8242](DataDog/dd-trace-java#8242) - [@richardstartin](https://github.com/richardstartin)) - 🐛 TempLocationManager Fixes and Improvements ([#8191](DataDog/dd-trace-java#8191) - [@jbachorik](https://github.com/jbachorik)) - ✨ Bump ddprof to 1.18.0 ([#8173](DataDog/dd-trace-java#8173) - [@jbachorik](https://github.com/jbachorik)) - ✨ Report profiler initialization and configuration errors to telemetry ([#8171](DataDog/dd-trace-java#8171) - [@jbachorik](https://github.com/jbachorik)) ##### Telemetry - ✨ Add pending traces report in tracer flares ([#8053](DataDog/dd-trace-java#8053) - [@mhlidd](https://github.com/mhlidd)) ##### Testing - ✨ Test http server requests in parallel ([#8222](DataDog/dd-trace-java#8222) - [@amarziali](https://github.com/amarziali)) ##### Trace context propagation - ✨ Add non default propagator registration ([#8310](DataDog/dd-trace-java#8310) - [@PerfectSlayer](https://github.com/PerfectSlayer)) ##### Tracer core - ✨ Probe for existence of IBMSASL or ACCP security providers ([#8276](DataDog/dd-trace-java#8276) - [@mcculls](https://github.com/mcculls)) - ✨⚡ Overhead improvement to agent feedback based sampling ([#8265](DataDog/dd-trace-java#8265) - [@dougqh](https://github.com/dougqh)) - 🧹 Move async propagation API from scope to tracer ([#8231](DataDog/dd-trace-java#8231) - [@PerfectSlayer](https://github.com/PerfectSlayer)) - ✨ Introduce context propagation API ([#8161](DataDog/dd-trace-java#8161) - [@PerfectSlayer](https://github.com/PerfectSlayer)) - ✨🧪 Use env-entry to add tags per webapp deployment ([#8138](DataDog/dd-trace-java#8138) - [@amarziali](https://github.com/amarziali)) - ✨ Introduce context helpers API ([#8134](DataDog/dd-trace-java#8134) - [@PerfectSlayer](https://github.com/PerfectSlayer)) - ✨ Support IPv6 values for `DD_AGENT_HOST` and `DD_TRACE_AGENT_URL` ([#7984](DataDog/dd-trace-java#7984) - [@mhlidd](https://github.com/mhlidd)) ##### Instrumentations ##### Apache HttpComponents - 🐛 Properly finish spans and support latest apache httpclient5 ([#8272](DataDog/dd-trace-java#8272) - [@amarziali](https://github.com/amarziali)) ##### AWS Lambda instrumentation - 🐛 Properly capture lambda payloads for all handler types. ([#8264](DataDog/dd-trace-java#8264) - [@purple4reina](https://github.com/purple4reina)) ##### AWS S3 instrumentation - 💡 Create S3 instrumentation + add span pointers ([#8075](DataDog/dd-trace-java#8075) - [@nhulston](https://github.com/nhulston)) ##### AWS SDK instrumentation - 🐛 Revert "Add avoid double instrumenting lambda non-streaming handlers." ([#8247](DataDog/dd-trace-java#8247) - [@nhulston](https://github.com/nhulston)) ##### Cassandra - ✨ Allow extracting keyspace from statement result ([#8239](DataDog/dd-trace-java#8239) - [@amarziali](https://github.com/amarziali)) ##### Core Java language instrumentation - ✨ Propagation of translateEscapes of String class ([#8186](DataDog/dd-trace-java#8186) - [@sezen-datadog](https://github.com/sezen-datadog)) ##### Eclipse Vert.x instrumentation - 🐛 Fix vertx worker propagation and error handling ([#8237](DataDog/dd-trace-java#8237) - [@amarziali](https://github.com/amarziali)) - ✨ Support vertx 5 ([#8220](DataDog/dd-trace-java#8220) - [@amarziali](https://github.com/amarziali)) - ✨ Add support for session tracking in Vertx ([#8167](DataDog/dd-trace-java#8167) - [@manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) ##### Kafka instrumentation - 🐛 Prevent possible NPE calculating Kafka record header size ([#8292](DataDog/dd-trace-java#8292) - [@ygree](https://github.com/ygree)) ##### Mule instrumentation - 🐛 Fix crash using Mule with JPMS ([#8187](DataDog/dd-trace-java#8187) - [@amarziali](https://github.com/amarziali)) ##### Protocol Buffer instrumentation - ✨ Change hash computation for protobuf to better represent impacting changes + save proto number in schema ([#8201](DataDog/dd-trace-java#8201) - [@vandonr](https://github.com/vandonr)) ##### Spring instrumentation - 🐛 Preserve getQualifier from spring scheduling runnables ([#8293](DataDog/dd-trace-java#8293) - [@amarziali](https://github.com/amarziali)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am every weekday" in timezone Australia/Melbourne, Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://github.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). GitOrigin-RevId: bb09d47e4eed77a003f630273b4d0a84003eb899

change hash computation for protobuf to be closer to impacting changes

9804100

vandonr requested review from a team as code owners January 14, 2025 16:35

vandonr requested a review from smola January 14, 2025 16:36

vandonr added inst: protobuf Protocol Buffer instrumentation type: enhancement comp: data streams Data Streams Monitoring labels Jan 14, 2025

add protobuf number as extension to the saved schema

34cf5f7

vandonr changed the title ~~Change hash computation for protobuf to better represent impacting changes~~ Change hash computation for protobuf to better represent impacting changes + save proto number in schema Jan 15, 2025

update test

d702a13

piochelepiotr approved these changes Jan 15, 2025

View reviewed changes

vandonr added comp: api Tracer public API and removed comp: api Tracer public API labels Jan 16, 2025

amarziali approved these changes Jan 16, 2025

View reviewed changes

vandonr merged commit 4025000 into master Jan 23, 2025
174 checks passed

vandonr deleted the vandonr/proto branch January 23, 2025 08:57

github-actions bot added this to the 1.46.0 milestone Jan 23, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Change hash computation for protobuf to better represent impacting changes + save proto number in schema #8201

Change hash computation for protobuf to better represent impacting changes + save proto number in schema #8201

vandonr commented Jan 14, 2025 •

edited

Loading

github-actions bot commented Jan 14, 2025 •

edited

Loading

pr-commenter bot commented Jan 14, 2025 •

edited

Loading

Change hash computation for protobuf to better represent impacting changes + save proto number in schema #8201

Change hash computation for protobuf to better represent impacting changes + save proto number in schema #8201

Conversation

vandonr commented Jan 14, 2025 • edited Loading

What Does This Do

Additional Notes

Contributor Checklist

github-actions bot commented Jan 14, 2025 • edited Loading

pr-commenter bot commented Jan 14, 2025 • edited Loading

Benchmarks

Startup

Parameters

Summary

Load

Parameters

Summary

Dacapo

vandonr commented Jan 14, 2025 •

edited

Loading

github-actions bot commented Jan 14, 2025 •

edited

Loading

pr-commenter bot commented Jan 14, 2025 •

edited

Loading