fix paths with trailing separator

k-yle / pdf-to-img

📃📸 Converts PDFs to images in nodejs

https://npm.im/pdf-to-img

MIT License

85 stars 22 forks source link

fix paths with trailing separator #195

Closed antlanc7 closed 1 year ago

antlanc7 commented 1 year ago

Hi, I am sorry because my last fix in #194 was not enough, I checked with the new 2.1.0 release and the bug is still present, the fact is that pdfjs doesn't use path.join to build the paths, so the trailing path separator at the end has to be added manually.

k-yle commented 1 year ago

Weird... Thanks for the PR. I've added your example pdf as a test case, which is currently passing, even without this PR. I can confirm the warning disappears as of this PR.

antlanc7 commented 1 year ago

With the new 2.1.1 release it's working fine. Thank you :)