-
-
Notifications
You must be signed in to change notification settings - Fork 225
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Absolute and some relative paths broken in 10.0.1 #303
Comments
I can confirm. |
I still experience the issue on windows for version @11.0.4 |
We also experience this issue still on windows. |
This is still not fixed and makes the library unusable on windows |
Same on macOS here! |
I believe this is fixed by setting |
Version 9.1.0 = OK
Version 10.0.1 = BROKEN
This is broken on Windows only.
In our code, we call this module like this:
$RefParser.dereference(path.resolve('swagger/index.yaml'), ...
This worked in version 9.1.0. In version 10.0.1, it returns the error:
ResolverError: Error opening file "__path_to_root__\\node_modules\\@apidevtools\\json-schema-ref-parser\\__path_to_root__\\swagger\\index.yaml"
In file
node_modules\@apidevtools\json-schema-ref-parser\cjs\util\projectDir.cjs
:projectDir
is returned as__path_to_root__\\node_modules\\@apidevtools\\json-schema-ref-parser
So to get our code to work, we have to do one of two things:
$RefParser.dereference(path.resolve('swagger/index.yaml'), ...
to$RefParser.dereference('../../../swagger/index.yaml', ...
node_modules\@apidevtools\json-schema-ref-parser\cjs\util\projectDir.cjs
, changeconst projectDir = nodePath.resolve(__dirname, "..", "..");
toconst projectDir = nodePath.resolve(__dirname, "..", "..", "..", "..", "..");
The text was updated successfully, but these errors were encountered: