Skip to content

Commit 522d154

Browse files
authored
Sender shorter version of host (#2680)
1 parent c7e8406 commit 522d154

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/statsbeat/NetworkStatsbeat.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ private static void addCommonProperties(
241241
StatsbeatTelemetryBuilder telemetryBuilder, IntervalMetricsKey key) {
242242
telemetryBuilder.addProperty("endpoint", BREEZE_ENDPOINT);
243243
telemetryBuilder.addProperty("cikey", key.getIkey());
244-
telemetryBuilder.addProperty("host", key.getHost());
244+
telemetryBuilder.addProperty("host", shorten(key.getHost()));
245245
String causeFieldName = key.getCauseFieldName();
246246
Object cause = key.getCauseValue();
247247
if (causeFieldName != null && cause != null) {
@@ -300,7 +300,7 @@ private double getRequestDurationAvg() {
300300
* e.g. endpointUrl 'https://westus-0.in.applicationinsights.azure.com/v2.1/track' host will
301301
* return 'westus-0'
302302
*/
303-
static String getHost(String endpointUrl) {
303+
static String shorten(String endpointUrl) {
304304
Matcher matcher = hostPattern.matcher(endpointUrl);
305305

306306
if (matcher.find()) {

agent/agent-tooling/src/test/java/com/microsoft/applicationinsights/agent/internal/statsbeat/NetworkStatsbeatTest.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -111,27 +111,27 @@ public void run() {
111111
@Test
112112
public void testGetHost() {
113113
String url = "https://fakehost-1.example.com/";
114-
assertThat(NetworkStatsbeat.getHost(url)).isEqualTo("fakehost-1");
114+
assertThat(NetworkStatsbeat.shorten(url)).isEqualTo("fakehost-1");
115115

116116
url = "https://fakehost-2.example.com/";
117-
assertThat(NetworkStatsbeat.getHost(url)).isEqualTo("fakehost-2");
117+
assertThat(NetworkStatsbeat.shorten(url)).isEqualTo("fakehost-2");
118118

119119
url = "http://www.fakehost-3.example.com/";
120-
assertThat(NetworkStatsbeat.getHost(url)).isEqualTo("fakehost-3");
120+
assertThat(NetworkStatsbeat.shorten(url)).isEqualTo("fakehost-3");
121121

122122
url = "http://www.fakehost.com/v2/track";
123-
assertThat(NetworkStatsbeat.getHost(url)).isEqualTo("fakehost");
123+
assertThat(NetworkStatsbeat.shorten(url)).isEqualTo("fakehost");
124124

125125
url = "https://www.fakehost0-4.com/";
126-
assertThat(NetworkStatsbeat.getHost(url)).isEqualTo("fakehost0-4");
126+
assertThat(NetworkStatsbeat.shorten(url)).isEqualTo("fakehost0-4");
127127

128128
url = "https://www.fakehost-5.com";
129-
assertThat(NetworkStatsbeat.getHost(url)).isEqualTo("fakehost-5");
129+
assertThat(NetworkStatsbeat.shorten(url)).isEqualTo("fakehost-5");
130130

131131
url = "https://fakehost.com";
132-
assertThat(NetworkStatsbeat.getHost(url)).isEqualTo("fakehost");
132+
assertThat(NetworkStatsbeat.shorten(url)).isEqualTo("fakehost");
133133

134134
url = "http://fakehost-5/";
135-
assertThat(NetworkStatsbeat.getHost(url)).isEqualTo("fakehost-5");
135+
assertThat(NetworkStatsbeat.shorten(url)).isEqualTo("fakehost-5");
136136
}
137137
}

0 commit comments

Comments
 (0)