search

browserify / crypto-browserify

partial implementation of node's `crypto` for the browser
MIT License
653 stars 199 forks source link

Support md4 #184

Open d3lm opened 5 years ago

d3lm commented 5 years ago

I am running into an issue where I need to support md4 hashes. Would it be an option to use this package to support md4?

calvinmetcalf commented 5 years ago

in the short term you probably want to use that package directly, I'm not sure if we want to add it in, and if we did how quickly it would go in

d3lm commented 5 years ago

How can I add this only for my use case? Can you help me out here? What do I need to do to support md4 in my case?

d3lm commented 5 years ago

BTW, thanks for your quick response, I really appreciate it.

calvinmetcalf commented 5 years ago

you can just directly require and use that library you linked to

d3lm commented 5 years ago

The problem is I am using a lib that uses the 'crypto' lib and passes in md4 as an algorithm and I have no control of that. So I am not using crypto myself but rather it's used by another lib which requires md4 to be supported.

rawr51919 commented 3 years ago

A polyfill may help with this particular usage case in your webpack config.

rawr51919 commented 3 years ago

Something tells me that this could in fact go in relatively easily in a PR (after #200 and #201, of course), however, the usage case, as I described before, would work best if it was polyfilled in your webpack config. It depends on the usage case.