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

fix authentication #316

Merged
merged 1 commit into from
Dec 7, 2024
Merged

fix authentication #316

merged 1 commit into from
Dec 7, 2024

Conversation

tux-mind
Copy link
Contributor

@tux-mind tux-mind commented Dec 4, 2024

Apple has introduced some dynamic GET parameters into their redirects, forcing us to use the main domain ( no p71- and such prefixes ) to obtain those parameters.

However, when following such redirects, we shall also increment the attempt parameter ( something that was hard-coded before ).

Apple has introduced some dynamic GET parameters into their
redirects, forcing us to use the main domain ( no `p71-`
and such prefixes ) to obtain those parameters.

However, when following such redirects, we shall also increment
the `attempt` parameter ( something that was hard-coded before ).
@mooring
Copy link

mooring commented Dec 5, 2024

Without the 2FA code, it works fine. Have you tested the two-step verification scenario?

I tested the two-step verification, and it prompted twice for a verification code. After entering the first code with an error response, it then showed the second prompted verification code, but the main process had exited with an error.

Is there something wrong?

image

@tux-mind
Copy link
Contributor Author

tux-mind commented Dec 5, 2024

@mooring have you used it with --auth-code xxxxxx ? It works on my end.
The only error I got was when I got a dialog on the device asking to allow the login from a new device, which is the expected behavior I guess.
I've also tried to auth revoke and re-auth and it works, but I guess that by using the same mac-address Apple caches the grant permission response.

@majd
Copy link
Owner

majd commented Dec 7, 2024

Great contribution! Thank you.

@majd majd merged commit 8a23d6f into majd:main Dec 7, 2024
8 of 10 checks passed
@tux-mind tux-mind mentioned this pull request Dec 10, 2024
2 tasks
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.

3 participants