Skip to content
This repository was archived by the owner on Nov 18, 2024. It is now read-only.

Wallet encryption - #79

Open
Fiono11 wants to merge 40 commits into
feeless:mainfrom
Fiono11:wallet-encryption
Open

Wallet encryption#79
Fiono11 wants to merge 40 commits into
feeless:mainfrom
Fiono11:wallet-encryption

Conversation

@Fiono11

@Fiono11 Fiono11 commented Apr 8, 2021

Copy link
Copy Markdown
Contributor

Issue #14 .

@gak
gak enabled auto-merge (squash) April 8, 2021 20:38
Comment thread 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
Copy Markdown
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
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👀

@gak

gak commented Apr 11, 2021

Copy link
Copy Markdown
Collaborator

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

Good to see they are responsive! 👍

@gak

gak commented May 21, 2021

Copy link
Copy Markdown
Collaborator

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

Geal commented Aug 21, 2021

Copy link
Copy Markdown

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

@gak

gak commented Aug 22, 2021

Copy link
Copy Markdown
Collaborator

Thank you @Geal :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants