- 🇺🇸United States angrytoast PNW
The MR code in https://git.drupalcode.org/project/openid_connect/-/merge_requests/41 works well. For context, this works well for our use case which initiates login from the idP (Okta, in this case) and it fixes a confusing experience where the user may have a Drupal session, but then come through the idP and get an error screen.
I applied this patch to openid_connect 3.0.0-alpha2, it works for me. Tests looks ok.
I don't think we need an option to be enable/disable the check as in the related issue, this one is good enough.
- last update
over 1 year ago 99 pass, 1 fail - Status changed to Needs work
8 months ago 7:16pm 6 November 2024 - 🇨🇦Canada chrisck BC, Canada
MR!41 does not apply to the 3.x-dev branch. Needs a re-roll.
- 🇮🇳India prem suthar Ahemdabad- Gujrat , Jodhpur - Rajsthan
prem suthar → made their first commit to this issue’s fork.
- 🇦🇺Australia lykyd
Reroll of patch comment #7, compatible for version `3.0.0-alpha5`
Changes need to be applied to the Merge Request, rather than as a patch.
- First commit to issue fork.
- @el7cosmos opened merge request.
- 🇮🇩Indonesia el7cosmos 🇮🇩 GMT+7
I made a new MR based on 3.x branch as I can't change the base branch on the existing MR