Require CookieBuilder.Name to be a not null or empty

Author: Nate McMaster

src/Microsoft.AspNetCore.Http.Abstractions/CookieBuilder.cs

@@ -2,6 +2,7 @@
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System;
+using Microsoft.AspNetCore.Http.Abstractions;
 
 namespace Microsoft.AspNetCore.Http
 {
@@ -10,10 +11,18 @@ namespace Microsoft.AspNetCore.Http
     /// </summary>
     public class CookieBuilder
     {
+        private string _name;
+
         /// <summary>
         /// The name of the cookie.
         /// </summary>
-        public virtual string Name { get; set; }
+        public virtual string Name
+        {
+            get => _name;
+            set => _name = !string.IsNullOrEmpty(value)
+                ? value
+                : throw new ArgumentException(Resources.ArgumentCannotBeNullOrEmpty, nameof(value));
+        }
 
         /// <summary>
         /// The cookie path.

src/Microsoft.AspNetCore.Http.Abstractions/Properties/Resources.Designer.cs

@@ -153,4 +153,7 @@
   <data name="Exception_UseMiddlewareExplicitArgumentsNotSupported" xml:space="preserve">
     <value>Types that implement '{0}' do not support explicit arguments.</value>
   </data>
+  <data name="ArgumentCannotBeNullOrEmpty" xml:space="preserve">
+    <value>Argument cannot be null or empty.</value>
+  </data>
 </root>