<\/p>\n
Un probl\u00e8me r\u00e9curent qui peut affecter divers fichiers d’un site web (page xHTML, CSS, script PHP…) concerne la pr\u00e9sence de caract\u00e8res blancs en amont et aval de tels fichiers. Diverses raisons peuvent mener au besoin de supprimer ces caract\u00e8res, d’o\u00f9 la r\u00e9alisation du script qui suit qui s’occupe automatiquement d’identifier r\u00e9cursivement les fichiers concern\u00e9s et de les traiter.<\/p>\n
Ces espaces blancs peuvent \u00eatre :<\/p>\n
Attention toutefois au \\0 qui est marqueur de fin de fichier pour certains.<\/p>\n
Mais pourquoi et comment j’en suis venu \u00e0 pondre ce petit script ?<\/p>\n
La premi\u00e8re raison concerne la compression des fichiers. Sur un serveur web, il y a tr\u00e8s souvent des fichiers qui ont un nombre incalculable d’espace ou de retour \u00e0 la ligne en amont ou en aval. Pourquoi les conserver? Sachant qu’\u00e0 chaque appel de la page il sont transmis au navigateur de l’utilisateur \u00e0 raison d’un octet par caract\u00e8re. \u00c7a fait du d\u00e9bit inutile et suppl\u00e9mentaire!<\/p>\n
La seconde raison est lors de la conversion d’encodage de fichier. C’est pr\u00e9cis\u00e9ment le probl\u00e8me que j’ai rencontr\u00e9. J’ai converti toute une base de donn\u00e9es, tous les fichiers et toute la configuration d’un serveur de l’ISO-8859-1 vers UTF-8. Que vous utilisiez “iconv” ou une autre solution de conversion, il se peut qu’un d\u00e9savantage survienne \u00e0 cette conversion : certains caract\u00e8res blancs en ISO sont doubl\u00e9s ou convertis en d’autres whitespace lors de cette conversion. Ce qui a engendr\u00e9 dans mon cas sur plusieurs milliers de fichiers l’ajout de plusieurs retour \u00e0 la ligne en amont ou en aval…<\/p>\n
La troisi\u00e8me raison, qui d\u00e9coule de la seconde, est que lorsqu’un fichier PHP a des whitespaces en amont ou en aval, ces caract\u00e8res sont directement transmis lors de leur interpr\u00e9tation au navigateur de l’utilisateur. Ainsi, dans le cas d’utilisation de session, donc avec un “session_start()” \u00e0 un certain endroit, il ne faut ABSOLUMENT PAS que des donn\u00e9es aient \u00e9t\u00e9 transf\u00e9r\u00e9es au pr\u00e9alable, sinon une erreur survient. Donc, tous les fichiers interpr\u00e9t\u00e9s ou inclus ne doivent pas effectuer d’echo ou afficher des caract\u00e8res html ou whitespace avant ce “session_start()”. D’o\u00f9 ce robot de nettoyage.<\/p>\n
Le probl\u00e8me des “sessions_start()” n’en est qu’un parmi plusieurs. Toutes g\u00e9n\u00e9rations en PHP de document\/fichier dynamique d’un autre type n\u00e9cessite qu’aucune donn\u00e9e ne soit transmise avant. C’est le cas de la g\u00e9n\u00e9ration d’image dynamique via GD2 ou m\u00eame de la g\u00e9n\u00e9ration de PDF dynamique avec FPDF. Ces g\u00e9n\u00e9rations sont faites en PHP et avant de les transmettre au client un en-t\u00eate avec un content-type MIME correcte doit \u00eatre d\u00e9fini, ce qui implique qu’aucune donn\u00e9es ne doit \u00eatre transmises au pr\u00e9alable.<\/p>\n
Bien s\u00fbr, il y a la solution de capturer le tampon avec des “ob_start()”, “ob_end()”, “ob_end_flush()”, “ob_get_contents()” et appliquer des “trim()” avant d’envoyer le tampon, mais c’est de la “triche” et des traitements suppl\u00e9mentaires c\u00f4t\u00e9 serveur…<\/p>\n
J’ai donc d\u00e9velopp\u00e9 le script suivant qui analyse r\u00e9cursivement un path absolu, v\u00e9rifie tous les fichiers, affiche en rouge ceux ayant des whistespace en amont ou aval, et supprime les whitespaces des fichiers s\u00e9lectionn\u00e9s apr\u00e8s confirmation sans toucher au reste du contenu du fichier.<\/p>\n
[php]&amp;lt;!DOCTYPE html PUBLIC &amp;quot;-\/\/W3C\/\/DTD XHTML 1.0 Strict\/\/EN&amp;quot; &amp;quot;http:\/\/www.w3.org\/TR\/xhtml1\/DTD\/xhtml1-transitional.dtd&amp;quot;&amp;gt;&lt;\/p&gt;<br \/>
\n&lt;p&gt;&amp;lt;html xmlns=&amp;quot;http:\/\/www.w3.org\/1999\/xhtml&amp;quot;&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;head&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;style&amp;gt;&lt;br \/&gt;<br \/>
\n *{&lt;br \/&gt;<br \/>
\n font-family:verdana;&lt;br \/&gt;<br \/>
\n font-size:10px;&lt;br \/&gt;<br \/>
\n }&lt;\/p&gt;<br \/>
\n&lt;p&gt; .error{&lt;br \/&gt;<br \/>
\n font-weight:bold;&lt;br \/&gt;<br \/>
\n color:#AA0000;&lt;br \/&gt;<br \/>
\n }&lt;\/p&gt;<br \/>
\n&lt;p&gt; .success{&lt;br \/&gt;<br \/>
\n font-weight:bold;&lt;br \/&gt;<br \/>
\n color:#00AA00;&lt;br \/&gt;<br \/>
\n }&lt;br \/&gt;<br \/>
\n &amp;lt;\/style&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;\/head&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;body&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;form action=&amp;quot;&amp;quot; method=&amp;quot;post&amp;quot;&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;table&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;tr&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;td&amp;gt;Absolute path to analyze :&amp;lt;\/td&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;td&amp;gt;&amp;lt;input type=&amp;quot;text&amp;quot; name=&amp;quot;path&amp;quot; size=&amp;quot;100&amp;quot; value=&amp;quot;&amp;lt;?php echo $_SERVER[&amp;quot;DOCUMENT_ROOT&amp;quot;];?&amp;gt;&amp;quot; \/&amp;gt;&amp;lt;\/td&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;\/tr&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;\/table&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;input type=&amp;quot;submit&amp;quot; value=&amp;quot;Run&amp;quot; \/&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;\/form&amp;gt;&lt;\/p&gt;<br \/>
\n&lt;p&gt;&amp;lt;?php&lt;\/p&gt;<br \/>
\n&lt;p&gt;function printFile($file, $blankExists){&lt;br \/&gt;<br \/>
\n echo &amp;quot;&amp;lt;li class=’&amp;quot; . (($blankExists) ? &amp;quot;error&amp;quot; : &amp;quot;success&amp;quot;) . &amp;quot;’&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;input type=’checkbox’ value=’$file’ name=’files[]’ &amp;quot; . (($blankExists) ? &amp;quot;checked&amp;quot; : &amp;quot;&amp;quot;) . &amp;quot;\/&amp;gt;&lt;br \/&gt;<br \/>
\n $file&lt;br \/&gt;<br \/>
\n &amp;lt;\/li&amp;gt;&amp;quot;;&lt;br \/&gt;<br \/>
\n}&lt;\/p&gt;<br \/>
\n&lt;p&gt;function analyzeFile($file){&lt;br \/&gt;<br \/>
\n $contents = @file_get_contents($file);&lt;br \/&gt;<br \/>
\n return (preg_match(&amp;quot;\/^(\\n|\\t|\\r| |\\x0B|\\&amp;#92;&amp;#48;)\/i&amp;quot;, $contents) || preg_match(&amp;quot;\/(\\n|\\t|\\r| |\\x0B|\\&amp;#92;&amp;#48;)$\/i&amp;quot;, $contents));&lt;br \/&gt;<br \/>
\n}&lt;\/p&gt;<br \/>
\n&lt;p&gt;function cleanFile($file){&lt;br \/&gt;<br \/>
\n $flag = false;&lt;br \/&gt;<br \/>
\n $contents = @file_get_contents($file);&lt;br \/&gt;<br \/>
\n $fp = @fopen($file, &amp;quot;w&amp;quot;);&lt;br \/&gt;<br \/>
\n if($fp){&lt;br \/&gt;<br \/>
\n @fputs($fp, trim($contents));&lt;br \/&gt;<br \/>
\n @fclose($fp);&lt;br \/&gt;<br \/>
\n $flag = true;&lt;br \/&gt;<br \/>
\n }&lt;br \/&gt;<br \/>
\n return $flag;&lt;br \/&gt;<br \/>
\n}&lt;\/p&gt;<br \/>
\n&lt;p&gt;function fileWalker($luke){&lt;br \/&gt;<br \/>
\n $dp = @opendir($luke) or die(&amp;quot;Directory $luke doesn’t exist…&amp;quot;);&lt;br \/&gt;<br \/>
\n $indexExists = false;&lt;br \/&gt;<br \/>
\n $folders = array();&lt;\/p&gt;<br \/>
\n&lt;p&gt; while($file = @readdir($dp)){&lt;br \/&gt;<br \/>
\n if(in_array($file, array(&amp;quot;.&amp;quot;, &amp;quot;..&amp;quot;, &amp;quot;.svn&amp;quot;))) continue;&lt;br \/&gt;<br \/>
\n if(is_dir($luke.’\/’.$file))&lt;br \/&gt;<br \/>
\n $folders[] = $luke.’\/’.$file;&lt;br \/&gt;<br \/>
\n else {&lt;br \/&gt;<br \/>
\n printFile($luke.’\/’.$file, analyzeFile($luke.’\/’.$file));&lt;br \/&gt;<br \/>
\n }&lt;br \/&gt;<br \/>
\n }&lt;br \/&gt;<br \/>
\n @closedir($dp);&lt;\/p&gt;<br \/>
\n&lt;p&gt; foreach($folders as $folder){&lt;br \/&gt;<br \/>
\n echo &amp;quot;&amp;lt;ul&amp;gt;&amp;quot;;&lt;br \/&gt;<br \/>
\n fileWalker($folder);&lt;br \/&gt;<br \/>
\n echo &amp;quot;&amp;lt;\/ul&amp;gt;&amp;quot;;&lt;br \/&gt;<br \/>
\n }&lt;\/p&gt;<br \/>
\n&lt;p&gt;}&lt;\/p&gt;<br \/>
\n&lt;p&gt;if(isset($_POST[&amp;quot;path&amp;quot;])){&lt;br \/&gt;<br \/>
\n echo &amp;quot;&amp;lt;form action=” method=’post’&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;input type=’submit’ value=’Trim files’ \/&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;ul&amp;gt;&amp;quot;;&lt;br \/&gt;<br \/>
\n fileWalker(trim(strval($_POST[&amp;quot;path&amp;quot;])));&lt;br \/&gt;<br \/>
\n echo &amp;quot; &amp;lt;\/ul&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;input type=’submit’ value=’Trim files’ \/&amp;gt;&lt;br \/&gt;<br \/>
\n &amp;lt;\/form&amp;gt;&amp;quot;;&lt;br \/&gt;<br \/>
\n} elseif(isset($_POST[&amp;quot;files&amp;quot;]) &amp;amp;&amp;amp; is_array($_POST[&amp;quot;files&amp;quot;]) &amp;amp;&amp;amp; count($_POST[&amp;quot;files&amp;quot;]) &amp;gt; 0){&lt;br \/&gt;<br \/>
\n foreach($_POST[&amp;quot;files&amp;quot;] as $file){&lt;br \/&gt;<br \/>
\n $file = trim(strval($file));&lt;br \/&gt;<br \/>
\n if(file_exists($file) &amp;amp;&amp;amp; !is_dir($file) &amp;amp;&amp;amp; is_writable($file)){&lt;br \/&gt;<br \/>
\n if(cleanFile($file)){&lt;br \/&gt;<br \/>
\n echo &amp;quot;&amp;lt;div class=’success’&amp;gt;$file&amp;lt;\/div&amp;gt;&amp;quot;;&lt;br \/&gt;<br \/>
\n } else&lt;br \/&gt;<br \/>
\n echo &amp;quot;&amp;lt;div class=’error’&amp;gt;$file&amp;lt;\/div&amp;gt;&amp;quot;;&lt;br \/&gt;<br \/>
\n }&lt;br \/&gt;<br \/>
\n }&lt;br \/&gt;<br \/>
\n}&lt;\/p&gt;<br \/>
\n&lt;p&gt;?&amp;gt;&lt;\/p&gt;<br \/>
\n&lt;p&gt;&amp;lt;\/body&amp;gt;&lt;br \/&gt;<br \/>
\n&amp;lt;\/html&amp;gt;[\/php]<\/p>\n
En esp\u00e9rant que \u00e7a puisse d\u00e9panner certains! \ud83d\ude09<\/p>\n<\/p>\n
<\/p>","protected":false},"excerpt":{"rendered":"
Un probl\u00e8me r\u00e9curent qui peut affecter divers fichiers d’un site web (page xHTML, CSS, script PHP…) concerne la pr\u00e9sence de […]<\/p>\n","protected":false},"author":1337,"featured_media":1195,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[16,15],"tags":[20,17,22,23,19,21,18],"class_list":["post-95","post","type-post","status-publish","format-standard","has-post-thumbnail","hentry","category-prog-and-dev","category-projects-and-tools","tag-iso-8859-1","tag-php","tag-recursivite","tag-sessions-php","tag-trim","tag-utf-8","tag-whitespace"],"_links":{"self":[{"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/posts\/95","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/users\/1337"}],"replies":[{"embeddable":true,"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/comments?post=95"}],"version-history":[{"count":11,"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/posts\/95\/revisions"}],"predecessor-version":[{"id":1633,"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/posts\/95\/revisions\/1633"}],"wp:featuredmedia":[{"embeddable":true,"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/media\/1195"}],"wp:attachment":[{"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/media?parent=95"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/categories?post=95"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.asafety.fr\/en\/wp-json\/wp\/v2\/tags?post=95"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}