setAttribute lijkt te worden genegeerd
Het werkt echt heel mooi, maar... ik merk dat setAttribute op lijn 11 wordt genegeerd.
Ik zie dat er in de img-tag die in de for-loop aangeroepen wordt wel netjes de URL wordt ge-replaced, maar er komt geen nieuw attribuut bij die 'data-original-src' heet?
Waarom negeert hij dit?
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
<?php
public function parse($string) {
if($this->settings['camo_enabled'] == true) {
$doc = new DOMDocument();
$doc->loadHTML($string);
$images = $doc->getElementsByTagName('img');
//$imgarray = array();
foreach ($images as $image) {
$url = $image->getAttribute('src');
if(substr($url, 0, 8) != 'https://') {
$image->setAttribute('data-original-src', $url);
$camo = $this->proxy_url($url);
$string = str_replace($image->getAttribute('src'), $camo, $string);
unset($url, $camo);
}
}
}
return $string;
}
?>
public function parse($string) {
if($this->settings['camo_enabled'] == true) {
$doc = new DOMDocument();
$doc->loadHTML($string);
$images = $doc->getElementsByTagName('img');
//$imgarray = array();
foreach ($images as $image) {
$url = $image->getAttribute('src');
if(substr($url, 0, 8) != 'https://') {
$image->setAttribute('data-original-src', $url);
$camo = $this->proxy_url($url);
$string = str_replace($image->getAttribute('src'), $camo, $string);
unset($url, $camo);
}
}
}
return $string;
}
?>
Gewijzigd op 21/09/2017 20:23:12 door - Ariën -
Quote:
This function returns a new instance of class DOMNodeList containing all the elements with a given local tag name.
Oftewel, je bent naar alle waarschijnlijkheid gegevens in een aparte kopie aan het wijzigen die alleen binnen de scope van de functie bestaat, het origineel blijft ongewijzigd.
Meerdere oplossingen mogelijk. Je zou bijvoorbeeld de functie in eerste instantie rechtstreeks het object (DOMDocument) kunnen voeren. Objecten zijn call by reference dus ben je daarmee direct in het origineel aan het werken.
Gewijzigd op 21/09/2017 22:16:45 door Thomas van den Heuvel
Maar getAttribute() werkt wel.
Het is puur dat setAttribute() niet werkt.
Ik zie het al, ik ga kijken of ik de image-tag opnieuw op kan bouwen.
Gewijzigd op 21/09/2017 22:31:04 door - Ariën -
Code (php)
1
2
2
$src = $image->getAttribute('src');
$string = str_replace($src, "$camo\" data-original-src=\"$src", $string);
$string = str_replace($src, "$camo\" data-original-src=\"$src", $string);
Een soort van HTML-injectie zeg maar (de bestaande src=" en sluit " recyclen).
Als de quootjes niet consequent zijn kun je evt nog met een regex aan de slag, maar dan wordt het al gauw nasty.
Toevoeging op 21/09/2017 22:51:50:
Gaat trouwens fout als hetzelfde plaatje (URL) 2x in de $string voorkomt
Die injectie ziet er wel kazig uit, inderdaad.