James Newton-King

Hi. I took a look and discovered that `Timer` is no longer needed. All targets now have System.Threading.Timer that can be used directly. The fundamental problem was the timer was...

Test on main failed with this error. > Aspire.Hosting.Elasticsearch.Tests.ElasticsearchFunctionalTests.Aspire.Hosting.Elasticsearch.Tests.ElasticsearchFunctionalTests.WithDataShouldPersistStateBetweenUsages(useVolume: False) ``` Aspire.Hosting.DistributedApplicationException : Application orchestrator dependency check returned an error: The operation has timed out. at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line...

That looks expected. If the dashboard server stops running while the browser is still open then the browser will eventually error because it can't contact the server.

This has been blocked for a while now. What's the latest status on this PR?

I don't know. I didn't design the API. I agree it's a limitation. What you could do is [access the underlying HttpContext](https://learn.microsoft.com/en-us/aspnet/core/grpc/aspnetcore?view=aspnetcore-8.0&tabs=visual-studio#resolve-httpcontext-in-grpc-methods) in the interceptor and add headers to `HttpContext.Response.Headers`.

I'm not familiar with the HTTP server log standard discussed here. For collecting log information, we have other existing options: Structured JSON console output for listening to standard console output,...

Parsing with commas is automatically supported by enum parse. Could that be used? ```cs string input = "Read, Write"; Permissions userPermissions = (Permissions) Enum.Parse(typeof(Permissions), input); Console.WriteLine(userPermissions); // Output: Read, Write...

The blank lines are unrelated.

The issues with logging are caused by how logging was originally designed: the host is always listening to logs from DCP and stores them in memory. That design was then...

Bug in DCP that didn't add timestamps to some logs is resolved. Will look at this next week.