@diosmosis opened this Pull Request on September 25th 2018 Member

Changes:

I am not sure if this fix will have repercussions so I haven't modified the tests. Will do so after someone else gives their opinion.

Fixes #13460

@tsteur commented on September 25th 2018 Member

If it fixes your issue and tests pass probably all good to go. It's almost impossible to review it since the segments are just a mess and we never know where they are urlencoded or urldecoded etc.

@diosmosis commented on September 25th 2018 Member

It will change a lot of the test output I think (another encoding in the segment property in API output). Think this won't be a BC break? at least not a potentially huge one.

@tsteur commented on September 25th 2018 Member

Think this won't be a BC break?

I really don't know. As long as we can still handle the segments urlencoded and urldecoded it should be fine.

@fdellwing commented on September 26th 2018 Contributor

Applied the patch locally and it resolves the issue for me.

Powered by GitHub Issue Mirror