Case-sensitivity fix for file extensions

Added .ToLower() to ensure extensions explicitly match those in SupportedMediaTypes.json
interfax · Mar 9, 2020 · 27d5059 · 27d5059
1 parent 1654506
commit 27d5059
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/InterFAX.Api/Documents.cs b/InterFAX.Api/Documents.cs
@@ -72,7 +72,7 @@ public IFaxDocument BuildFaxDocument(Uri fileUri)
         /// <returns></returns>
         public IFaxDocument BuildFaxDocument(byte[] file, string extension)
         {
-            var ext = extension.Trim('.');
+            var ext = extension.Trim('.').ToLower();
             var mediaType = SupportedMediaTypes.Keys.Contains(ext)
                 ? SupportedMediaTypes[ext]
                 : "application/octet-stream";
@@ -89,7 +89,7 @@ public IFaxDocument BuildFaxDocument(string filePath)
                 throw new FileNotFoundException(filePath);
 
             var extension = Path.GetExtension(filePath) ?? "*";
-            extension = extension.TrimStart('.');
+            extension = extension.TrimStart('.').ToLower();
 
             var mediaType = SupportedMediaTypes.Keys.Contains(extension)
                 ? SupportedMediaTypes[extension]
@@ -104,7 +104,7 @@ public IFaxDocument BuildFaxDocument(string filePath)
         public IFaxDocument BuildFaxDocument(string fileName, FileStream fileStream)
         {
             var extension = Path.GetExtension(fileName) ?? "*";
-            extension = extension.TrimStart('.');
+            extension = extension.TrimStart('.').ToLower();;
 
             var mediaType = SupportedMediaTypes.Keys.Contains(extension)
                 ? SupportedMediaTypes[extension]