Add support for external url as the user login form

Actually, 8.x branch was opened, lets make sure this lands there.

Guys,

follow the patch for Drupal 8.
I implemented the hook_user_login the redirect aflter login.

Fixed patch

Correct file

The new version.

From 8.x-1.x

New version.
Esse class not found fixed

Version more clean

I reviewed the patch and here are some improvements you could do if this method goes through:

+  $destination = \Drupal::request()->getQueryString();
+  parse_str($destination, $output);

parse_str will fail on null values, and that's a perfectly valid return value for getQueryString().
You could use \Drupal::request()->get() or \Drupal::request()->query->get() instead as it's easier to handle and always works.

This way of solving this problem is also weird, seems like a partial fix on top of a partial fix.
You should probably be solving this on r4032login_redirect() - which apparently is not doing what it's supposed to.

Most likely because the code was not re-evaluated during the D8 port.

@fidelix

I changed the way of resolving this. Now, in the onKernelException method, I verify if its possible to convert the destination from absolute to relative

Since the patch was actually obsolete (edited 2 years ago), I rewrote it to match the current code.

It's now committed so the module support now external url.

Thanks.