phc-argon2 icon indicating copy to clipboard operation
phc-argon2 copied to clipboard

Merge into argon2

Open ranisalt opened this issue 3 years ago • 2 comments

Hello! argon2 maintainer here. Since we adopted @phc/format a couple of years back, argon2 and @phc/argon2 have pretty much the same interface (argon2 exports a bit more stuff, but compatible). I was thinking @phg/argon2 should be merged into argon2, with argon2 adopting whatever is missing so that importing @phc/argon2 or argon2 is compatible. Then, you could just proxy the library (export * from 'argon2') and we would just have one library. What do you say?

ranisalt avatar Mar 22 '22 10:03 ranisalt

The phc-* project started as a way to provide a unified API for all the password hasing algoritmhs (mainly to build https://github.com/simonepri/upash).

I did not work on this for a long time now, so I'm unsure what's the state and how many people are using this packages (It seems almost none, https://www.npmjs.com/package/@phc/argon2).

As far as I remember, my idea was that say for argon2:

  • argon2 is the implementation, exposing whatever API they like
  • @phc/argon2 is the public API, not subject to major changes over time In my view of things I immagined the two packages to be maintained by the same maintainer, but of course this never happened.

Just to help me understand this better, what's the benefit in merging?

simonepri avatar Jun 04 '22 16:06 simonepri

Sorry for the massive delay, must have missed the notification

Just to help me understand this better, what's the benefit in merging?

Not much more than simplifying this package. argon2 and @phc/argon2 export the same functions and this will very likely not change, given that Argon2 and PHC format go hand in hand. It would also cover this:

In my view of things I immagined the two packages to be maintained by the same maintainer

ranisalt avatar Jan 31 '23 09:01 ranisalt