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

Wallet encryption #79

Open
wants to merge 40 commits into
base: main
Choose a base branch
from
Open

Wallet encryption #79

wants to merge 40 commits into from

Conversation

Fiono11
Copy link
Contributor

@Fiono11 Fiono11 commented Apr 8, 2021

Issue #14 .

@gak gak enabled auto-merge (squash) April 8, 2021 20:38
src/errors.rs Outdated
DecryptionError(#[from] age::DecryptError),
//#[error("Error encrypting file")]
//EncryptionError(#[from] age::EncryptError),
// TODO: this is giving an error, but I don't understand why
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's this one, @gak .

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀

@gak
Copy link
Collaborator

gak commented Apr 11, 2021

I guess we're waiting for str4d/rage#221 ?

Good to see they are responsive! 👍

@gak
Copy link
Collaborator

gak commented May 21, 2021

Blocked by rust-bakery/nom#1311

auto-merge was automatically disabled June 9, 2021 14:42

Head branch was pushed to by a user without write access

@Geal
Copy link

Geal commented Aug 21, 2021

you can now use nom 7, which fixes the issue with bitvec

@gak
Copy link
Collaborator

gak commented Aug 22, 2021

Thank you @Geal :)

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

Successfully merging this pull request may close these issues.

None yet

3 participants