Please see https://github.com/matomo-org/matomo/issues/16057 for details on what this commit is fixing.
Well, this is big for me since without this patch I couldn't login ;-)
That's why I took the effort to fix it.
On Tue, Jun 16, 2020, 21:32 Thomas Steur <firstname.lastname@example.org> wrote:
Generally thinking because there are some minor changes in there that
could break something we might prefer to have this in Matomo 4 and not
Matomo 3 where we now only fix security issues and bigger bugs.
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
@luismsgomes sorry about that. There will be only very few releases for 3.X in the future so it might be worth patching it in the worst case. Maybe we could think of a less "risky" version for 3.X where it doesn't change as much. Eg for the case where a port is defined like in https://github.com/matomo-org/matomo/pull/16058/files#diff-e3d1b64ec2cd9e138ad8d5c1a0d09a0eR140 we are now no longer returning the URLs without port. This could be causing trouble to other users maybe (as it's always hard to tell with these things)
@Findus23 can you think of any objections or do you reckon it should be fine?
I know little about how this part works, so I can't comment on it.