Skip to content

Commit 9f38abe

Browse files
RSam25Release Workflow
andauthored
Implemented optimized HTTPathMetricFilter (#2270)
* Precomputed pathSet * Created a more optimized version of HTTPPathMetricFilter * Removed original import statement from shared * Moved new filter path function to shared * [CI Pipeline] Released Snapshot version: 5.63.5-alpha-279-SNAPSHOT * Updated vulnerable urllib in aws scripts * Reverted Dockerfile and renamed constants * Updated shared version --------- Co-authored-by: Release Workflow <unifiedid-admin+release@thetradedesk.com>
1 parent 747b4cb commit 9f38abe

File tree

5 files changed

+8
-6
lines changed

5 files changed

+8
-6
lines changed

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,4 +30,4 @@ CMD java \
3030
-Djava.security.egd=file:/dev/./urandom \
3131
-Dvertx.logger-delegate-factory-class-name=io.vertx.core.logging.SLF4JLogDelegateFactory \
3232
-Dlogback.configurationFile=/app/conf/logback.xml \
33-
-jar ${JAR_NAME}-${JAR_VERSION}.jar
33+
-jar ${JAR_NAME}-${JAR_VERSION}.jar

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
<enclave-aws.version>2.1.0</enclave-aws.version>
2424
<enclave-azure.version>2.1.19</enclave-azure.version>
2525
<enclave-gcp.version>2.1.9</enclave-gcp.version>
26-
<uid2-shared.version>11.1.124</uid2-shared.version>
26+
<uid2-shared.version>11.4.0</uid2-shared.version>
2727
<image.version>${project.version}</image.version>
2828
<maven.compiler.source>21</maven.compiler.source>
2929
<maven.compiler.target>21</maven.compiler.target>

scripts/aws/requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
requests[socks]==2.32.3
22
boto3==1.35.59
3-
urllib3==2.6.0
3+
urllib3==2.6.3
44
PyYAML===6.0.2

src/main/java/com/uid2/operator/Main.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,7 @@ private static void setupMetrics(MicrometerMetricsOptions metricOptions) {
517517
// providing common renaming for prometheus metric, e.g. "hello.world" to "hello_world"
518518
.meterFilter(new PrometheusRenameFilter())
519519
.meterFilter(MeterFilter.replaceTagValues(Label.HTTP_PATH.toString(),
520-
actualPath -> HTTPPathMetricFilter.filterPath(actualPath, Endpoints.pathSet())))
520+
actualPath -> HTTPPathMetricFilter.filterPathWithoutPathParameters(actualPath, Endpoints.pathSet())))
521521
// Don't record metrics for 404s.
522522
.meterFilter(MeterFilter.deny(id ->
523523
id.getName().startsWith(MetricsDomain.HTTP_SERVER.getPrefix()) &&

src/main/java/com/uid2/operator/vertx/Endpoints.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,16 @@ public enum Endpoints {
2929
UID2_SDK_1_0_0("/static/js/uid2-sdk-1.0.0.js"),
3030
UID2_SDK_2_0_0("/static/js/uid2-sdk-2.0.0.js")
3131
;
32-
private final String path;
32+
private static final Set<String> ENDPOINTS_SET = Stream.of(Endpoints.values()).map(Endpoints::toString).collect(Collectors.toSet());
33+
34+
private final String path;
3335

3436
Endpoints(final String path) {
3537
this.path = path;
3638
}
3739

3840
public static Set<String> pathSet() {
39-
return Stream.of(Endpoints.values()).map(Endpoints::toString).collect(Collectors.toSet());
41+
return ENDPOINTS_SET;
4042
}
4143

4244
@Override

0 commit comments

Comments
 (0)