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

adds support for the new login procedure #93

Merged
merged 1 commit into from
Jan 15, 2024
Merged

Conversation

Gezzo42
Copy link
Contributor

@Gezzo42 Gezzo42 commented Jan 9, 2024

@Gezzo42
Copy link
Contributor Author

Gezzo42 commented Jan 9, 2024

One check failed, because setuptools is used, according to pyca/cryptography#6411
Do you have any idea how to solve this?

@hthiery
Copy link
Owner

hthiery commented Jan 10, 2024

Hi, thanks for providing this PR. Unfortunatly I do not know how to solve this issue. One possible workaround could be to disable the pypy support or check if a newer version of pypy (e.g. 3.10) changes the behavior.

@hthiery
Copy link
Owner

hthiery commented Jan 10, 2024

Hi, thanks for providing this PR. Unfortunatly I do not know how to solve this issue. One possible workaround could be to disable the pypy support or check if a newer version of pypy (e.g. 3.10) changes the behavior.

I did a test and your changes seems to build with pypy3.9 and pypy3.10. So please change the used version in .github/workflows/test.yml.

While on it please meld/rebase your first 3 commits into one.

* FRITZ!OS 7.24 and later supports new response generation using PBKDF2
* technical note: https://avm.de/fileadmin/user_upload/Global/Service/Schnittstellen/AVM%20Technical%20Note%20-%20Session%20ID_EN%20-%20Nov2020.pdf
* adds test for login using new variant
* updates to pypy3.9 due to build problems with pypy3.7
@coveralls
Copy link

Coverage Status

coverage: 93.059% (+0.04%) from 93.02%
when pulling 1c8d5c1 on Gezzo42:master
into 53afd61 on hthiery:master.

@hthiery
Copy link
Owner

hthiery commented Jan 15, 2024

thanks

@hthiery hthiery merged commit 08661c7 into hthiery:master Jan 15, 2024
12 checks passed
@mib1185
Copy link
Collaborator

mib1185 commented Jan 15, 2024

may this break for older Fritz!Box?

@hthiery
Copy link
Owner

hthiery commented Jan 15, 2024

may this break for older Fritz!Box?

As I understand this should have a fallback to old login mechanism: 1c8d5c1#diff-2739f308d4808e84f7bc38e336077eaaf3c76dbe979f853522a499c66d872391R127

@mib1185
Copy link
Collaborator

mib1185 commented Jan 15, 2024

ah ok 👍

@Gezzo42
Copy link
Contributor Author

Gezzo42 commented Jan 15, 2024

may this break for older Fritz!Box?

As I understand this should have a fallback to old login mechanism: 1c8d5c1#diff-2739f308d4808e84f7bc38e336077eaaf3c76dbe979f853522a499c66d872391R127

Correct point. It should work. I could not test it, because my Fritz!Box firmware is newer. But the old test for it is still active and working correctly.

@Gezzo42
Copy link
Contributor Author

Gezzo42 commented Jan 15, 2024

Thanks for merging :-)!

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.

4 participants