- Issue created by @au_dave
- 🇳🇱Netherlands Klaas Culemborg
Hi @au_dave, thanks for the patch. I work fine on my drupal site.
- First commit to issue fork.
I was struggling to integrate this module with Anonymous Login and Redirect After Login in combination with the site being in a subdirectory. I also had to retain the prefixing "/" from the destination in the session, but came up with a slightly alternate approach in the redirect controller that let me seamlessly integrate with Redirect After Login by distinguishing between a user-provided destination and a default destination (to do this, I set this module's login destination to nothing / empty string).
I previously put a comment on a Redirect After Login issue for compatibility with this module, which details more of the things I had to work through and why/how I came up with this solution, in case it's of interest/use: https://www.drupal.org/project/redirect_after_login/issues/3513324#comme... ✨ Improve compatibility with OpenID Connect destination Active .
A potential benefit of this approach to set the destination in the query is that it doesn't require copying Drupal core code from RedirectResponseSubscriber (because it will be triggered by use of the query destination).