Skip to content

Commit 87ecab8

Browse files
committed
fix: cannot use endpoint to knwon ip
1 parent a5f655e commit 87ecab8

File tree

5 files changed

+24
-15
lines changed

5 files changed

+24
-15
lines changed

projects/RabbitMQ.Client/Impl/RabbitMQActivitySource.cs

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -205,6 +205,15 @@ internal static void SetNetworkTags(this Activity? activity, IFrameHandler frame
205205
{
206206
if (activity?.IsAllDataRequested ?? false)
207207
{
208+
switch (frameHandler.RemoteEndPoint.AddressFamily)
209+
{
210+
case AddressFamily.InterNetworkV6:
211+
activity.SetTag("network.type", "ipv6");
212+
break;
213+
case AddressFamily.InterNetwork:
214+
activity.SetTag("network.type", "ipv4");
215+
break;
216+
}
208217
activity.SetServerTags(frameHandler.Endpoint);
209218

210219
if (frameHandler.RemoteEndPoint is IPEndPoint ipEndpoint)
@@ -235,16 +244,6 @@ internal static void SetNetworkTags(this Activity? activity, IFrameHandler frame
235244

236245
internal static void SetServerTags(this Activity activity, AmqpTcpEndpoint endpoint)
237246
{
238-
switch (endpoint.AddressFamily)
239-
{
240-
case AddressFamily.InterNetworkV6:
241-
activity.SetTag("network.type", "ipv6");
242-
break;
243-
case AddressFamily.InterNetwork:
244-
activity.SetTag("network.type", "ipv4");
245-
break;
246-
}
247-
248247
if (!string.IsNullOrEmpty(endpoint.HostName))
249248
{
250249
activity

projects/RabbitMQ.Client/PublicAPI.Unshipped.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,3 @@ RabbitMQ.Client.Exceptions.PublishReturnException.ReplyCode.get -> ushort
66
RabbitMQ.Client.Exceptions.PublishReturnException.ReplyText.get -> string!
77
RabbitMQ.Client.Exceptions.PublishReturnException.RoutingKey.get -> string!
88
const RabbitMQ.Client.RabbitMQActivitySource.ConnectionSourceName = "RabbitMQ.Client.Connection" -> string!
9-
static RabbitMQ.Client.EndpointResolverExtensions.SelectOneAsync<T>(this RabbitMQ.Client.IEndpointResolver! resolver, System.Func<RabbitMQ.Client.AmqpTcpEndpoint!, System.Threading.CancellationToken, System.Threading.Tasks.Task<T>!>! selector, System.Threading.CancellationToken cancellationToken) -> System.Threading.Tasks.Task<(T, System.Diagnostics.Activity?)>!

projects/Test/Integration/Integration.csproj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22

33
<PropertyGroup Condition="$([MSBuild]::IsOSPlatform('Windows'))">
4-
<TargetFrameworks>net8.0;net472</TargetFrameworks>
4+
<TargetFrameworks>net9.0;net472</TargetFrameworks>
55
<NoWarn>$(NoWarn);CA2007</NoWarn>
66
<TreatWarningsAsErrors>true</TreatWarningsAsErrors>
77
</PropertyGroup>
88

99
<PropertyGroup Condition="!$([MSBuild]::IsOSPlatform('Windows'))">
10-
<TargetFramework>net8.0</TargetFramework>
10+
<TargetFramework>net9.0</TargetFramework>
1111
<NoWarn>$(NoWarn);CA2007</NoWarn>
1212
<TreatWarningsAsErrors>true</TreatWarningsAsErrors>
1313
</PropertyGroup>

projects/Test/Integration/TestConnectionFactory.cs

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -447,7 +447,18 @@ public async Task TestCreateConnectionRegisterAnActivity()
447447
ConnectionFactory cf = CreateConnectionFactory();
448448
await using IConnection conn = await cf.CreateConnectionAsync();
449449
var connectionActivity = connectionRecorder.VerifyActivityRecordedOnce();
450+
connectionActivity.HasTag("network.peer.address");
451+
connectionActivity.HasTag("network.local.address");
452+
connectionActivity.HasTag("server.address");
453+
connectionActivity.HasTag("client.address");
454+
connectionActivity.HasTag("network.peer.port");
455+
connectionActivity.HasTag("network.local.port");
456+
connectionActivity.HasTag("server.port");
457+
connectionActivity.HasTag("client.port");
458+
connectionActivity.HasTag("network.type");
450459
var tcpConnectionActivity = tcpConnectionRecorder.VerifyActivityRecordedOnce();
460+
tcpConnectionActivity.HasTag("server.port");
461+
tcpConnectionActivity.HasTag("server.address");
451462
Assert.Equal(connectionActivity, tcpConnectionActivity.Parent);
452463
await conn.CloseAsync();
453464
}

projects/Test/SequentialIntegration/SequentialIntegration.csproj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22

33
<PropertyGroup Condition="$([MSBuild]::IsOSPlatform('Windows'))">
4-
<TargetFrameworks>net8.0;net472</TargetFrameworks>
4+
<TargetFrameworks>net9.0;net472</TargetFrameworks>
55
<NoWarn>$(NoWarn);CA2007</NoWarn>
66
<TreatWarningsAsErrors>true</TreatWarningsAsErrors>
77
</PropertyGroup>
88

99
<PropertyGroup Condition="!$([MSBuild]::IsOSPlatform('Windows'))">
10-
<TargetFramework>net8.0</TargetFramework>
10+
<TargetFramework>net9.0</TargetFramework>
1111
<NoWarn>$(NoWarn);CA2007</NoWarn>
1212
<TreatWarningsAsErrors>true</TreatWarningsAsErrors>
1313
</PropertyGroup>

0 commit comments

Comments
 (0)