Skip to content

Commit d2d72e3

Browse files
Copilotstephentoub
andcommitted
Remove unused usingOuterActivity parameter from AddTags method
The parameter was only used for DisplayName setting which was removed as redundant. Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
1 parent 40ad4c9 commit d2d72e3

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

src/ModelContextProtocol.Core/McpSessionHandler.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,7 @@ public async Task<JsonRpcResponse> SendRequestAsync(JsonRpcRequest request, Canc
467467
{
468468
if (addTags)
469469
{
470-
AddTags(ref tags, activity, request, method, target, usingOuterActivity);
470+
AddTags(ref tags, activity, request, method, target);
471471
}
472472

473473
if (_logger.IsEnabled(LogLevel.Trace))
@@ -657,9 +657,6 @@ private static string GetMethodName(JsonRpcMessage message) =>
657657
};
658658

659659
private void AddTags(ref TagList tags, Activity? activity, JsonRpcMessage message, string method, string? target)
660-
=> AddTags(ref tags, activity, message, method, target, usingOuterActivity: false);
661-
662-
private void AddTags(ref TagList tags, Activity? activity, JsonRpcMessage message, string method, string? target, bool usingOuterActivity)
663660
{
664661
tags.Add("mcp.method.name", method);
665662
tags.Add("network.transport", _transportKind);

0 commit comments

Comments
 (0)