Do BOM stripping for remote URL's

ChangeLog.md

@@ -1,3 +1,9 @@
+### 06/26/2018 0.4.51
+- Bug fix: Do BOM stripping for remote URL's [#266](https://github.com/Azure/oav/issues/266).
+
+### 06/20/2018 0.4.50
+- Replace  sway dependency with yasway.
+
 ### 06/19/2018 0.4.49
 - Bug fix: Data is valid against more than one schema from `oneOf` [#248](https://github.com/Azure/oav/pull/248)
   The problem occurs when referenced model may also accept `null`. The fix is replacing `oneOf` with `anyOf`.

lib/util/utils.ts

@@ -75,7 +75,7 @@ export async function parseJson(specPath: string): Promise<Spec> {
   } else {
     // local filepath
     try {
-      const fileContent = stripBOM(fs.readFileSync(specPath, "utf8"))
+      const fileContent = fs.readFileSync(specPath, "utf8")
       const result = parseContent(specPath, fileContent)
       docCache[specPath] = Promise.resolve(result)
       return result
@@ -101,10 +101,11 @@ export async function parseJson(specPath: string): Promise<Spec> {
  */
 export function parseContent(filePath: string, fileContent: string): Spec {
   let result = null
+  const sanitizedContent = stripBOM(fileContent)
   if (/.*\.json$/ig.test(filePath)) {
-    result = JSON.parse(fileContent)
+    result = JSON.parse(sanitizedContent)
   } else if (/.*\.ya?ml$/ig.test(filePath)) {
-    result = YAML.safeLoad(fileContent)
+    result = YAML.safeLoad(sanitizedContent)
   } else {
     const msg =
       `We currently support "*.json" and "*.yaml | *.yml" file formats for validating swaggers.\n` +

package.json

@@ -1,6 +1,6 @@
 {
   "name": "oav",
-  "version": "0.4.50",
+  "version": "0.4.51",
   "author": {
     "name": "Microsoft Corporation",
     "email": "[email protected]",