- Issue created by @rajeshreeputra
- Merge request !62Resolve #3533328 "Fix: Unable to connect the site through Acquia Connector." → (Merged) created by rajeshreeputra
- First commit to issue fork.
- 🇺🇸United States japerry KVUO
accounts.acquia.com is no longer available. Please see https://acquia.my.site.com/s/article/Acquia-ID-Implementation-and-Setup
We did some work back in December (#3491727) in anticipation of the Acquia ID rollout, but unfortunately the error we expected to get from accounts is not what matches what you're seeing here. Therefore, with all customer facing accounts now moved to Acquia ID, we need to remove all of the OpenID connect code, due to the new IDP not accepting static client IDs anymore.
- 🇮🇳India ajits India
@japerry thank you for the reply and for updating the priority! I also have a support request open for this issue.
- 🇮🇳India rajeshreeputra Pune
With information provided in #5, I have removed the latest commit. Requesting review.
- 🇮🇳India ajits India
@rajeshreeputra - The information provided in #5 was not about the commit I pushed afaict. It was about the bug we've encountered. I might be wrong. Can you please share what part of my commit you think was incorrect?
On another note, directly dropping a commit from a merge request doesn't seem correct. I suggest preserving the working history of the branch withgit revert
instead. - 🇺🇸United States japerry KVUO
Rajeshreeputra is correct here, we needed to simply adjust the post headers so the authenticate method would work. The client ID is no longer valid, so all of that code that you were working on is not needed anymore.
I've updated the MR to remove all of the oauth authorization token information. Note, for BC (because many of these methods are public), I've left most method signatures alone. In theory, you can run cron still and it'll get a token using the API key/secret provided during the login screen.
-
japerry →
committed ec2468e6 on 4.x authored by
rajeshreeputra →
Issue #3533328 by rajeshreeputra, japerry: Unable to connect the site...
-
japerry →
committed ec2468e6 on 4.x authored by
rajeshreeputra →
-
japerry →
committed 6a2d340b on 7.x-4.x authored by
rajeshreeputra →
Issue #3533328 by rajeshreeputra, japerry: Unable to connect the site...
-
japerry →
committed 6a2d340b on 7.x-4.x authored by
rajeshreeputra →
Automatically closed - issue fixed for 2 weeks with no activity.