Problem/Motivation

#3030651: Support URL-initiated Log-in through Provider removed the usage of ::getRedirectUrl() and ::getUrlOptions() methods and instead just duplicates the same logic inside ::authorize().

This also removed the possibility to alter redirect_uri without overriding the whole ::authorize() function.

See https://git.drupalcode.org/project/openid_connect/-/commit/546d5ec377701...

Issue fork openid_connect-3317308

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

tuutti created an issue. See original summary.

tuutti’s picture

tuutti commented
Category: Task » Bug report
Status: Active » Needs review

tuutti opened merge request !60

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Indeed, seems like the code wasn't really well adapted to the changes in the module between when the patch was first created and when it got committed.

jcnventura’s picture

jcnventura
Primary language English
commented

  • jcnventura committed 57df041 on 3.x authored by tuutti 
    Issue #3317308 by tuutti, jcnventura: OpenIdConnectClientBase::authorize...

  • jcnventura committed 1b2aab6 on 2.x authored by tuutti 
    Issue #3317308 by tuutti, jcnventura: OpenIdConnectClientBase::authorize...
jcnventura’s picture

jcnventura
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.