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.

Sign up for GitHub

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

Jump to bottom

Script isn't working because of missing parts #1

Open
Behike opened this issue Jan 9, 2025 · 3 comments · May be fixed by #3
Open

Script isn't working because of missing parts #1

Behike opened this issue Jan 9, 2025 · 3 comments · May be fixed by #3

Comments

@Behike
Copy link

Behike commented Jan 9, 2025

Hi!
First of all, thanks for your script, I have been successfully able to manually decrypt my Home Assistant backups with it. However, it seems that you have inadvertently inserted some removed some parts of your code.

ha-decrypt-backup-tool/decrypt_backup.py

Line 160 in a1a660d

if re.match(r'^([A-Z0-9]{4}-){6}[A-Z0-9]{4}

The regex is not finished.

And here:

ha-decrypt-backup-tool/decrypt_backup.py

Line 204 in a1a660d

, manual_key):

The beginning of the condition has been deleted.

In case someone else find your script, it would be great if those issues are fixed.
Thanks!
@BliXem1
Copy link

BliXem1 commented Jan 9, 2025

+1

@alkissack
Copy link

@Behike re: have been successfully able to manually decrypt my Home Assistant backups with it. Can you explain how please

@AndyTempel AndyTempel mentioned this issue Jan 10, 2025
Closed
Behike added a commit to Behike/ha-decrypt-backup-tool that referenced this issue Jan 11, 2025
@Behike
Fix Script isn't working because of missing parts cogneato#1
3e121ff 
Remove condition which breaks the script if kit isn't present but doesn't let manual entry
@Behike Behike linked a pull request Jan 11, 2025 that will close this issue
Open
@Behike
Copy link
Author

Behike commented Jan 11, 2025 •
edited
Loading

I've made a PR (which you @alkissack can use as a working script, let me know if you need more help) as I think that AndyTempel's PR makes too much modification compared to what this issue was initially made for.
I've also removed a condition which would wrongly stop the script if the emergency kit isn't present, instead of letting the user manually enter it.
However, all versions should not be working on Windows (at least on mine) because of the extractall commands which seem to be confused by the Linux archive opened on a Windows system (I get an error "seeking backwards is not allowed"), I couldn't find a quick fix for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@alkissack @Behike @BliXem1