Accessing a link with a destination parameter

I'm not sure if this is the right approach, but if we add a condition to check if there was an existing destination parameter, we can keep it and it will be URL-encoded so that it comes through well.

Sounds like it would take care of the problem.

But maybe all URL parameters should be URL-encoded, so they get passed to the right place; not just destination. In my case, destination was causing issue, but other parameters could cause other kinds of problems.

I'm not really familiar with this module, just exploring it before I decide to use it. It seems to me that any other existing query string is concatenated into the $destination variable that I wrapped in urlencode(). See: https://git.drupalcode.org/issue/r4032login-3441317/-/blob/2.2.x/src/Eve...