raml-js-parser icon indicating copy to clipboard operation
raml-js-parser copied to clipboard

Fixed issue where paths with URL encoded characters were being passed…

Open drb opened this issue 10 years ago • 3 comments

… to the filesystem when attempting to load a file using fetchLocalFileAsync(), causing an exception to be raised.

Using a path like path/to a file/file.raml would be passed to the method as path/to%20a%20file/file.raml which would raise an exception from the call to fs.readFile. A simple patch has been added to normalize urlencoded strings before being passed into this method.

drb avatar Aug 12 '15 12:08 drb

Can you add a test to verify the regression?

dmartinezg avatar Aug 12 '15 13:08 dmartinezg

Also just to double, check, have you signed the Contributor agreement?

dmartinezg avatar Aug 12 '15 13:08 dmartinezg

I'll add a test ASAP. Contrib agreement is signed!

drb avatar Aug 12 '15 14:08 drb