Skip to content

Commit 539a5a9

Browse files
authored
Move ServerStarted telemetry after initialization (#905)
* Move server started telemetry * Add CHANGELOG
1 parent 2f7a9ab commit 539a5a9

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

core/Azure.Mcp.Core/src/Areas/Server/Commands/ServiceStartCommand.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,10 +130,11 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
130130

131131
await InitializeServicesAsync(host.Services);
132132

133+
await host.StartAsync(CancellationToken.None);
134+
133135
var telemetryService = host.Services.GetRequiredService<ITelemetryService>();
134136
LogStartTelemetry(telemetryService, options);
135137

136-
await host.StartAsync(CancellationToken.None);
137138
await host.WaitForShutdownAsync(CancellationToken.None);
138139

139140
return context.Response;

servers/Azure.Mcp.Server/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ The Azure MCP Server updates automatically by default whenever a new release com
1212

1313
### Bugs Fixed
1414
- Convert ARG usage to .NET SDK method calls of GetEntraAdministratorsAsync and ListFirewallRulesAsync to resolve the related issues ([#779](https://github.com/microsoft/mcp/issues/779) and [#855](https://github.com/microsoft/mcp/issues/855)) in `azmcp_entra_administrator_list` and `azmcp_sql_server_firewall_rule_list` commands. [[#891](https://github.com/microsoft/mcp/pull/891)]
15+
- Fixed `ServerStarted` telemetry event not being published. [[#905](https://github.com/microsoft/mcp/pull/905)]
1516

1617
### Other Changes
1718

0 commit comments

Comments
 (0)