A lot of iframe embed URLs are now scheme-independent. Here is an example:

.

Relative Paths to Absolute URLs module converts src="//www.youtube.com/embed/8OJ7ih8EE7s" to src="/www.youtube.com/embed/8OJ7ih8EE7s". If the relative path starts with "//" instead of "/", this module should not change it.

CommentFileSizeAuthor
#2 rel_to_abs-test_double_slash-2341793-2.patch1.34 KBjeffschuler

Comments

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Version: 7.x-1.x-dev » 8.x-1.x-dev

Please, fix it in 8.x branch for ability to get into 7.x as backport

jeffschuler’s picture

jeffschuler
Location Boulder, Colorado
commented
Version: 8.x-1.x-dev » 2.2.1
Status: Active » Needs review
StatusFileSize
new rel_to_abs-test_double_slash-2341793-2.patch1.34 KB

This was already fixed in #2079001: Network path reference urls (//) should not be prefixed.

There's a check:
&& substr($txt, $pos, 2) != '//' to ensure the string doesn't start with "//".

The attached patch adds a simple case to the module's functional test to verify this.

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Version: 2.2.1 » 2.2.2
Status: Needs review » Fixed

tnx

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.