Skip to content

Commit 1a14851

Browse files
authored
Rename server.socket.address to network.peer.address (#93255)
1 parent 715293e commit 1a14851

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

src/libraries/System.Net.Http/src/System/Net/Http/SocketsHttpHandler/Metrics/ConnectionMetrics.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,18 +14,18 @@ internal sealed class ConnectionMetrics
1414
private readonly object _schemeTag;
1515
private readonly object _hostTag;
1616
private readonly object? _portTag;
17-
private readonly object? _socketAddressTag;
17+
private readonly object? _peerAddressTag;
1818
private bool _currentlyIdle;
1919

20-
public ConnectionMetrics(SocketsHttpHandlerMetrics metrics, string protocolVersion, string scheme, string host, int? port, string? socketAddress)
20+
public ConnectionMetrics(SocketsHttpHandlerMetrics metrics, string protocolVersion, string scheme, string host, int? port, string? peerAddress)
2121
{
2222
_metrics = metrics;
2323
_openConnectionsEnabled = _metrics.OpenConnections.Enabled;
2424
_protocolVersionTag = protocolVersion;
2525
_schemeTag = scheme;
2626
_hostTag = host;
2727
_portTag = port;
28-
_socketAddressTag = socketAddress;
28+
_peerAddressTag = peerAddress;
2929
}
3030

3131
// TagList is a huge struct, so we avoid storing it in a field to reduce the amount we allocate on the heap.
@@ -42,9 +42,9 @@ private TagList GetTags()
4242
tags.Add("server.port", _portTag);
4343
}
4444

45-
if (_socketAddressTag is not null)
45+
if (_peerAddressTag is not null)
4646
{
47-
tags.Add("server.socket.address", _socketAddressTag);
47+
tags.Add("network.peer.address", _peerAddressTag);
4848
}
4949

5050
return tags;

src/libraries/System.Net.Http/tests/FunctionalTests/MetricsTest.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,9 @@ protected static void VerifyTag<T>(KeyValuePair<string, object?>[] tags, string
4747
}
4848
}
4949

50-
private static void VerifySocketAddress(KeyValuePair<string, object?>[] tags)
50+
private static void VerifyPeerAddress(KeyValuePair<string, object?>[] tags)
5151
{
52-
string ipString = (string)tags.Single(t => t.Key == "server.socket.address").Value;
52+
string ipString = (string)tags.Single(t => t.Key == "network.peer.address").Value;
5353
IPAddress ip = IPAddress.Parse(ipString);
5454
Assert.True(ip.Equals(IPAddress.Loopback.MapToIPv6()) ||
5555
ip.Equals(IPAddress.Loopback) ||
@@ -122,7 +122,7 @@ protected static void VerifyOpenConnections(string actualName, object measuremen
122122
VerifySchemeHostPortTags(tags, uri);
123123
VerifyTag(tags, "network.protocol.version", GetVersionString(protocolVersion));
124124
VerifyTag(tags, "http.connection.state", state);
125-
VerifySocketAddress(tags);
125+
VerifyPeerAddress(tags);
126126
}
127127

128128
protected static void VerifyConnectionDuration(string instrumentName, object measurement, KeyValuePair<string, object?>[] tags, Uri uri, Version? protocolVersion)
@@ -132,7 +132,7 @@ protected static void VerifyConnectionDuration(string instrumentName, object mea
132132
Assert.InRange(value, double.Epsilon, 60);
133133
VerifySchemeHostPortTags(tags, uri);
134134
VerifyTag(tags, "network.protocol.version", GetVersionString(protocolVersion));
135-
VerifySocketAddress(tags);
135+
VerifyPeerAddress(tags);
136136
}
137137

138138
protected static void VerifyTimeInQueue(string instrumentName, object measurement, KeyValuePair<string, object?>[] tags, Uri uri, Version? protocolVersion, string method = "GET")

0 commit comments

Comments
 (0)