Skip to content

Commit f98bfe9

Browse files
arkocclaude
andcommitted
Accept host:port or scheme://host:port for TemporalServerHost
Changes TemporalServerHost from Uri to string so bare "host:port" values (as used in Temporal Cloud config) bind correctly. Aspire-style "scheme://host:port" is still handled via URI parsing fallback. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 6b11ab4 commit f98bfe9

2 files changed

Lines changed: 8 additions & 3 deletions

File tree

csharp/src/Infrastructure.DependencyInjection/TrainSolverBuilderExtensions.cs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public static TrainSolverBuilder AddTrainSolver(this IServiceCollection services
4242

4343
private static IServiceCollection AddTemporalWorkerClient(
4444
this IServiceCollection services,
45-
Uri serverHost,
45+
string serverHost,
4646
string @namespace,
4747
string? apiKey)
4848
{
@@ -53,9 +53,14 @@ private static IServiceCollection AddTemporalWorkerClient(
5353
PropertyNameCaseInsensitive = true,
5454
}), new DefaultFailureConverter());
5555

56+
// Accepts either "host:port" or "scheme://host:port" (Aspire passes the latter)
57+
var targetHost = Uri.TryCreate(serverHost, UriKind.Absolute, out var uri) && !string.IsNullOrEmpty(uri.Host)
58+
? $"{uri.Host}:{uri.Port}"
59+
: serverHost;
60+
5661
services.AddTemporalClient(otps =>
5762
{
58-
otps.TargetHost = $"{serverHost.Host}:{serverHost.Port}";
63+
otps.TargetHost = targetHost;
5964
otps.Namespace = @namespace;
6065
otps.DataConverter = dataConverter;
6166

csharp/src/Infrastructure.DependencyInjection/TrainSolverOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ public class TrainSolverOptions
44
{
55
public const string SectionName = "TrainSolver";
66

7-
public Uri TemporalServerHost { get; set; } = null!;
7+
public string TemporalServerHost { get; set; } = null!;
88

99
public string TemporalNamespace { get; set; } = "default";
1010

0 commit comments

Comments
 (0)