Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
If you request name/mail then the return method changes from GET to POST and then the error state is not correctly checked.
Steps to reproduce
Try to log in with Apple, then cancel. You get an error: "BadMethodCallException: Required parameter not passed: "code"
Proposed resolution
Use $request->get() instead of $request->query->get(). Note that there's no has() equivalent, so I changed the has() to get. While that is generally not recommended in favor of explicit checks, I think here it is fine.
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#2 | social_auth_apple-3202748-2.patch | 1.01 KB | Berdir |
Comments
Comment #2
BerdirComment #3
ayalon CreditAttribution: ayalon at Liip commentedThanks for the patch. I see the issue and fixed it.
Comment #7
ayalon CreditAttribution: ayalon at Liip commentedComment #8
ayalon CreditAttribution: ayalon at Liip commented