fix: Use just the media type to detect JSON, not whole datacontenttype

Fixes #216

Signed-off-by: Jon Skeet <[email protected]>

Author: jskeet

src/CloudNative.CloudEvents.NewtonsoftJson/JsonEventFormatter.cs

@@ -364,7 +364,7 @@ protected virtual void DecodeStructuredModeDataBase64Property(JToken dataBase64T
         /// <returns>The data to populate in the <see cref="CloudEvent.Data"/> property.</returns>
         protected virtual void DecodeStructuredModeDataProperty(JToken dataToken, CloudEvent cloudEvent)
         {
-            if (IsJsonMediaType(cloudEvent.DataContentType!))
+            if (IsJsonMediaType(new ContentType(cloudEvent.DataContentType!).MediaType))
             {
                 cloudEvent.Data = dataToken;
             }

src/CloudNative.CloudEvents.SystemTextJson/JsonEventFormatter.cs

@@ -377,7 +377,7 @@ protected virtual void DecodeStructuredModeDataBase64Property(JsonElement dataBa
         /// <returns>The data to populate in the <see cref="CloudEvent.Data"/> property.</returns>
         protected virtual void DecodeStructuredModeDataProperty(JsonElement dataElement, CloudEvent cloudEvent)
         {
-            if (IsJsonMediaType(cloudEvent.DataContentType!))
+            if (IsJsonMediaType(new ContentType(cloudEvent.DataContentType!).MediaType))
             {
                 cloudEvent.Data = dataElement.Clone();
             }

test/CloudNative.CloudEvents.UnitTests/NewtonsoftJson/JsonEventFormatterTest.cs

@@ -767,6 +767,7 @@ public void DecodeStructuredModeMessage_NonJsonContentType_JsonStringToken(strin
         [Theory]
         [InlineData(null)]
         [InlineData("application/json")]
+        [InlineData("application/json; charset=utf-8")]
         public void DecodeStructuredModeMessage_JsonContentType_JsonStringToken(string contentType)
         {
             var obj = CreateMinimalValidJObject();
@@ -785,6 +786,7 @@ public void DecodeStructuredModeMessage_JsonContentType_JsonStringToken(string c
         [InlineData(null)]
         [InlineData("application/json")]
         [InlineData("application/xyz+json")]
+        [InlineData("application/xyz+json; charset=utf-8")]
         public void DecodeStructuredModeMessage_JsonContentType_NonStringValue(string contentType)
         {
             var obj = CreateMinimalValidJObject();

test/CloudNative.CloudEvents.UnitTests/SystemTextJson/JsonEventFormatterTest.cs

@@ -787,6 +787,7 @@ public void DecodeStructuredModeMessage_NonJsonContentType_JsonStringToken(strin
         [Theory]
         [InlineData(null)]
         [InlineData("application/json")]
+        [InlineData("application/json; charset=utf-8")]
         public void DecodeStructuredModeMessage_JsonContentType_JsonStringToken(string contentType)
         {
             var obj = CreateMinimalValidJObject();
@@ -805,6 +806,7 @@ public void DecodeStructuredModeMessage_JsonContentType_JsonStringToken(string c
         [InlineData(null)]
         [InlineData("application/json")]
         [InlineData("application/xyz+json")]
+        [InlineData("application/xyz+json; charset=utf-8")]
         public void DecodeStructuredModeMessage_JsonContentType_NonStringValue(string contentType)
         {
             var obj = CreateMinimalValidJObject();