Merge pull request #122 from p1c2u/fix/file-handler-windows-compatibility-fix

File handler Windows compatibility fix

Author: p1c2u

.github/workflows/python-test.yml

@@ -10,10 +10,11 @@ on:
 
 jobs:
   test:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
     strategy:
       matrix:
         python-version: [2.7, 3.5, 3.6, 3.7, 3.8, 3.9]
+        os: [windows-latest, ubuntu-latest]
       fail-fast: false
     steps:
     - uses: actions/checkout@v2
@@ -27,7 +28,9 @@ jobs:
         pip install -r requirements.txt
         pip install -r requirements_dev.txt
         pip install -e .
+      shell: bash
     - name: Test
       run: python setup.py test
+      shell: bash
     - name: Upload coverage
       uses: codecov/codecov-action@v1

openapi_spec_validator/handlers/file.py

@@ -3,6 +3,7 @@
 from yaml import load
 
 from openapi_spec_validator.handlers.base import BaseHandler
+from openapi_spec_validator.handlers.utils import uri_to_path
 from openapi_spec_validator.loaders import ExtendedSafeLoader
 
 
@@ -19,12 +20,12 @@ def __call__(self, f):
 class FileHandler(FileObjectHandler):
     """OpenAPI spec validator file path handler."""
 
-    def __call__(self, f):
-        if isinstance(f, StringIO):
-            return super(FileHandler, self).__call__(f)
+    def __call__(self, uri):
+        if isinstance(uri, StringIO):
+            return super(FileHandler, self).__call__(uri)
 
-        assert f.startswith("file")
+        assert uri.startswith("file")
 
-        filename = f[7:]
-        with open(filename) as fh:
+        filepath = uri_to_path(uri)
+        with open(filepath) as fh:
             return super(FileHandler, self).__call__(fh)

openapi_spec_validator/handlers/utils.py

@@ -0,0 +1,12 @@
+import os.path
+
+from six.moves.urllib.parse import urlparse, unquote
+from six.moves.urllib.request import url2pathname
+
+
+def uri_to_path(uri):
+    parsed = urlparse(uri)
+    host = "{0}{0}{mnt}{0}".format(os.path.sep, mnt=parsed.netloc)
+    return os.path.normpath(
+        os.path.join(host, url2pathname(unquote(parsed.path)))
+    )