[MDL-60337] htmlpurifier incorrectly encodes domain names like http://teßt.de/ - Moodle Tracker

XML

Word

Printable

Affected Branches:
MOODLE_32_STABLE, MOODLE_33_STABLE
Fixed Branches:
MOODLE_34_STABLE
Pull from Repository:
git://github.com/marinaglancy/moodle.git
Pull Master Branch:
wip-~~MDL-60337~~-master
Pull Master Diff URL:
https://github.com/marinaglancy/moodle/compare/d2800e5928...wip-MDL-60337-master
Testing Instructions:
Hide

As a student post on the forum with a link to http://teßt.de

Check that your link directs to http://xn--tet-6ka.de and not to http://tesst.de

Copy the link http://teßt.de into the browser URL and make sure that it actually goes to the same site as http://xn--tet-6ka.de
Show
As a student post on the forum with a link to http://te ßt.de Check that your link directs to http://xn--tet-6ka.de and not to http://tesst.de Copy the link http://te ßt.de into the browser URL and make sure that it actually goes to the same site as http://xn--tet-6ka.de

added unittest to demostrate it

        $text = '<a href="http://teßt.de">teßt.de</a>';

        $expected = '<a href="http://xn--tet-6ka.de">teßt.de</a>';

        $this->assertSame($expected, purify_html($text));

Failed asserting that two strings are identical.

--- Expected

+++ Actual

@@ @@

-<a href="http://xn--tet-6ka.de">teßt.de</a>

+<a href="http://tesst.de">teßt.de</a>

has a non-specific relationship to

MDL-60279 Prepare Moodle for PHP 7.2

has been marked as being related by

MDL-60335 curlsecurityblockedhosts does not recognise faß.de as valid domain

Participants:: Adrian Greeve, Andrew Lyons, cameron1729, CiBoT, Eloy Lafuente (stronk7), Marina Glancy

Component watchers:: Adrian Greeve, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo