Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can the salsa20 family be pushed to crates.io? #15

Closed
sseemayer opened this issue Jun 23, 2019 · 13 comments
Closed

Can the salsa20 family be pushed to crates.io? #15

sseemayer opened this issue Jun 23, 2019 · 13 comments

Comments

@sseemayer
Copy link
Contributor

Similar to #2, I am migrating away from rust-crypto and would like to use the Salsa20 and ChaCha20 implementations contained in stream-ciphers. Would it be possible to push them to crates.io?

@sseemayer sseemayer changed the title Can the salsa20 family be pushed to crates.io Can the salsa20 family be pushed to crates.io? Jun 23, 2019
@tarcieri
Copy link
Member

I think it's ok to publish as is, although I think it'd be nice to find some way to factor it into salsa20 and chacha20 crates.

@sseemayer
Copy link
Contributor Author

I would be willing to give that refactoring a stab if given some guidance on what is desired.

Salsa20 and ChaCha20 share SalsaFamilyState and SalsaFamilyCipher. Do you suggest to have this struct and trait in a salsa20-family crate that will be re-exported from a salsa20 and chacha20 crate?

@tarcieri
Copy link
Member

That's one possible approach, yes

@newpavlov
Copy link
Member

I've registered salsa20-core crate, which we can use for common functionality.

@tarcieri
Now you should have access to salsa20-core, salsa20 and chacha20 crates, so you can upload crates yourself.

@tarcieri
Copy link
Member

@newpavlov excellent, thanks

@sseemayer
Copy link
Contributor Author

Wow, that's a quick response. Would you like me to submit a PR or do this refactoring yourselves?

@newpavlov
Copy link
Member

I think it will be faster for you to create a PR. Also we always welcome new contributors. :)

@sseemayer
Copy link
Contributor Author

Great, I'll get right to it!

@sseemayer
Copy link
Contributor Author

Would you like to keep the salsa20-family crate as a meta-crate that bundles salsa20 and chacha20 crates?

@tarcieri
Copy link
Member

Probably not. It's just a name we picked in the course of the PR.

@newpavlov
Copy link
Member

I don't think we need a meta-crate, in my experience they are rarely used in practice. So lets use only 3 crates. BTW while you at it, please update table in README with the added crates.

@sseemayer
Copy link
Contributor Author

OK, I think the PR #16 is ready for review now.

@tarcieri
Copy link
Member

tarcieri commented Jun 24, 2019

Now that #16 is merged, I have published (and pushed tags for) the following releases:

Thanks @emc2 for the implementation work and @sseemayer for the refactoring work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants