Harden the PathString converter

Author: HaoK

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

@@ -459,9 +459,20 @@ public static implicit operator string(PathString path)
 
     internal class PathStringConverter : TypeConverter
     {
+        public override bool CanConvertFrom(ITypeDescriptorContext context, Type sourceType)
+            => sourceType == typeof(string) 
+            ? true 
+            : base.CanConvertFrom(context, sourceType);
+
         public override object ConvertFrom(ITypeDescriptorContext context, CultureInfo culture, object value)
-        {
-            return new PathString((string)value);
-        }
+            => value is string 
+            ? new PathString((string)value) 
+            : base.ConvertFrom(context, culture, value);
+
+        public override object ConvertTo(ITypeDescriptorContext context,
+           CultureInfo culture, object value, Type destinationType)
+            => destinationType == typeof(string) 
+            ? value.ToString() 
+            : base.ConvertTo(context, culture, value, destinationType);
     }
 }

test/Microsoft.AspNetCore.Http.Abstractions.Tests/PathStringTests.cs

@@ -208,11 +208,18 @@ public void ToUriComponentEscapeCorrectly(string category, string input, string
         }
 
         [Fact]
-        public void PathStringConvertsFromString()
+        public void PathStringConvertsOnlyToAndFromString()
         {
             var converter = TypeDescriptor.GetConverter(typeof(PathString));
             PathString result = (PathString)converter.ConvertFromInvariantString("/foo");
             Assert.Equal("/foo", result.ToString());
+            Assert.Equal("/foo", converter.ConvertTo(result, typeof(string)));
+            Assert.True(converter.CanConvertFrom(typeof(string)));
+            Assert.False(converter.CanConvertFrom(typeof(int)));
+            Assert.False(converter.CanConvertFrom(typeof(bool)));
+            Assert.True(converter.CanConvertTo(typeof(string)));
+            Assert.False(converter.CanConvertTo(typeof(int)));
+            Assert.False(converter.CanConvertTo(typeof(bool)));
         }
     }
 }