search

raml-org / raml-js-parser

(deprecated) A RAML parser based on PyYAML written in CoffeScript and available for use as NodeJs module or in-browser.
195 stars 53 forks source link

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

Open drb opened 9 years ago

drb commented 9 years ago

… 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.

dmartinezg commented 9 years ago

Can you add a test to verify the regression?

dmartinezg commented 9 years ago

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

drb commented 9 years ago

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