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

add challenge/response #19

Closed
wants to merge 3 commits into from
Closed

Conversation

steadfasterX
Copy link
Contributor

this is based on PR #12 with the following difference:

instead of adding a --unlock option to lglaf.py I use an own executable.
The reason is that within download mode you need to authenticate
only ONCE ( - IF you need to) and afterwards you're able to exec every
other action like partitions.py etc without doing it again (until you reboot
into download mode again ofc).

this is based on PR #12 with the following difference:

instead of adding a --unlock option to lglaf.py I use an own executable.
The reason is that within download mode you need to authenticate
only ONCE ( - IF you need to) and afterwards you're able to exec every
other action like partitions.py etc without doing it again (until you reboot
into download mode again ofc).
@Lekensteyn
Copy link
Owner

The review comments from the other PR still applies, e.g. python 3 compat. Maybe consider giving @snoremaster3000 credit in the header of the new script for the discoveries?

@snoremaster3000
Copy link

snoremaster3000 commented Sep 21, 2017

Hey, I think I based my script on @joeblowma 's C version. I remember doing some reversing on it but I wasn't able to get the complete algorithm and he/she shared so I went ahead and just translated it (kind of poorly too, I'm not an experienced coder). So I think @joeblowma should get the credit.

Edit: Just looked through my inbox and this is definite the case, @joeblowma reversed it.
Thanks,

@Lekensteyn
Copy link
Owner

This got taken care of in #27, if there are any issues please open a new issue.

@Lekensteyn Lekensteyn closed this Nov 27, 2017
@steadfasterX steadfasterX deleted the pr_auth branch January 16, 2019 17:04
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