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

[feature] crypto.hash_file #275

Closed
jabbate19 opened this issue Aug 15, 2023 · 3 comments · Fixed by #276
Closed

[feature] crypto.hash_file #275

jabbate19 opened this issue Aug 15, 2023 · 3 comments · Fixed by #276
Labels
feature New feature or request

Comments

@jabbate19
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Eldritch doesn't have a way to hash a file

Describe the solution you'd like
crypto.hash_file(src, method) -> String

Most of the times I've had to hash in Rust I wrote it myself. There should be code in https://github.com/jabbate19/steeloxide somewhere

@jabbate19 jabbate19 added the feature New feature or request label Aug 15, 2023
@hulto
Copy link
Collaborator

hulto commented Aug 15, 2023

How would you feel about hashing variables instead that way we can use file.read and then sys.hash()

Might want its own module for hash, encrypt, compress, encode.

@jabbate19
Copy link
Collaborator Author

We can make a different method for running crypto on data thats not a file imo

@hulto
Copy link
Collaborator

hulto commented Aug 15, 2023

We can make a different method for running crypto on data thats not a file imo

Works for me.

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

Successfully merging a pull request may close this issue.

2 participants