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

adding an identity backup and auto load if corruption #1198

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

V0r-T3x
Copy link

@V0r-T3x V0r-T3x commented Dec 24, 2023

Description

A backup (marked with the '.original' extension) is created when the keys and fingerprint are generated. If the files are corrupted, the backups are loaded instead of a regenerate it. The pwnagotchi is protected.

Motivation and Context

In some crash cases, the key can be corrupted and the keys need to be generated again.
Pwnagotchi's identity protection commit

  • I have raised an issue to propose this change (required)

How Has This Been Tested?

It's already integrated since a while into fancygotchi. It served me well for all the crashed I caused while I developed my new features. I voluntary corrupted the key and I booted the pwnagotchi. The backupwas loaded directly on boot. (the bad 'laoding' type was fixed in the second commit.

Capture d'écran 2023-12-24 100716

It's now integrated to the jay's fork too

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I've read the CONTRIBUTION guide
  • I have signed-off my commits with git commit -s

The code make a backup of the key and fingerprint, and if those files are corrupted, the backup is loaded instead of re-generating the key. The pwnagotchi identity is protected in case of crash.
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

1 participant