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

Initial changes for FIFA 21 #19

Closed
wants to merge 2 commits into from
Closed

Initial changes for FIFA 21 #19

wants to merge 2 commits into from

Conversation

kaurikk
Copy link

@kaurikk kaurikk commented Oct 15, 2020

Changes:

  • Different URLs changed to ea.com
  • Updated game year variables to 21
  • Changed order for Persona request (requires now additional "Nucleus-Access-Code" header to be sent)
  • Changed URL for config.json request
  • Changed URL for accountinfo request

Note: i also changed MFA from E-mail to Phone app, this can be ignored i guess.

@kaurikk kaurikk mentioned this pull request Oct 15, 2020
@@ -325,6 +330,7 @@ public function login(?string $code = null) : array
'authCode' => $auth_code,
'redirectUrl' => 'nucleus:rest',
],
//"ds" => "3acf525ee688016f4a8e5e5ebe307a0f4bf400e6b33e34d0ecc057358820c9ac/cc39"
Copy link
Author

@kaurikk kaurikk Oct 15, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be something new, and feels like without that we get back 403 for that request. No idea how to generate that.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done, but unfortunately I do not know how it works myself. Without that you can't log in?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without this param doesn't matter. You got 403 because the auth_code need get again before this request.

@kaurikk kaurikk closed this by deleting the head repository Nov 22, 2022
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