Uppercase MD5 is now deprecated. FDF module should reference lowercase md5

pdffillerjs / pdffiller

Take an existing PDF Form and data and PDF Filler will create a new PDF with all given fields populated.

MIT License

286 stars 113 forks source link

Uppercase MD5 is now deprecated. FDF module should reference lowercase md5 #21

Closed ryates8 closed 8 years ago

ryates8 commented 8 years ago

https://www.npmjs.com/package/MD5

The index.js file within the fdf module references the deprecated version of md5. Not a big issue, but wanted to put it on your radar. We don't commit our node_modules so we had to request that everyone on the team fix this in their local copy's.

whitef0x0 commented 8 years ago

I'll look into this tommorow.

johntayl commented 8 years ago

It looks like the FDF module requires, but never uses MD5. See Here

whitef0x0 commented 8 years ago

Maybe we should just move the XFDF instead? (see my most recent issue)

johntayl commented 8 years ago

Closing. See #22