foliojs / font-manager

A C++ module for Node.js providing access to the system font catalog.
MIT License
295 stars 100 forks source link

Added sanity checks to handle non-compliant fonts #35

Closed attackgoat closed 4 years ago

attackgoat commented 5 years ago

Hello!

This section of code was failing in our application because of the crazy font files our users have.

I added these checks because the platform-specific code paths did not set all of these values for every record.

Hope this helps!

attackgoat commented 5 years ago

Future folks: It appears this repo has this fix plus node 12: https://www.npmjs.com/package/fontmanager-redux