Skip to content

Commit 39f6d3e

Browse files
committed
Use ThrowIfNullOrEmpty in Resource ctor to reject empty names
1 parent ac8add0 commit 39f6d3e

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/Aspire.Hosting.Azure.WebPubSub/AzureWebPubSubExtensions.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,9 +185,9 @@ public static IResourceBuilder<AzureWebPubSubHubResource> AddHub(this IResourceB
185185
{
186186
hubResource = new AzureWebPubSubHubResource(name, hubName, builder.Resource);
187187
builder.Resource.Hubs[hubName] = hubResource;
188+
return builder.ApplicationBuilder.AddResource(hubResource);
188189
}
189-
var hubBuilder = builder.ApplicationBuilder.CreateResourceBuilder(hubResource);
190-
return hubBuilder;
190+
return builder.ApplicationBuilder.CreateResourceBuilder(hubResource);
191191
}
192192

193193
/// <summary>

src/Aspire.Hosting/ApplicationModel/Resource.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public abstract class Resource : IResource
2727
/// <param name="name">The name of the resource.</param>
2828
protected Resource(string name)
2929
{
30-
ArgumentNullException.ThrowIfNull(name);
30+
ArgumentException.ThrowIfNullOrEmpty(name);
3131

3232
Name = name;
3333
}

0 commit comments

Comments
 (0)