Répondre au commentaire

Méfiez vous surtout des URL absolues ! :) (bis, ter, et fini après)

Submitted by pounard on ven, 07/04/2008 - 13:57

Après avoir fait un patch très moche, m'être fait incendier par le mainteneur du module, j'ai décidé de réfléchir plus et d'introspecter plus loin pour voir d'où venait mes problèmes. Et j'ai trouvé, voici l'explication en couleurs !:

file_create_url() à tout cassé, parce que:

<?php
function file_create_url($path) {
  // Strip file_directory_path from $path. We only include relative paths in urls.
  if (strpos($path, file_directory_path() . '/') === 0) {
    $path = trim(substr($path, strlen(file_directory_path())), '\\/');
  }
  switch (variable_get('file_downloads', FILE_DOWNLOADS_PUBLIC)) {
    case FILE_DOWNLOADS_PUBLIC:
      return $GLOBALS['base_url'] .'/'. file_directory_path() .'/'. str_replace('\\', '/', $path);
    case FILE_DOWNLOADS_PRIVATE:
      return url('system/files/'. $path, NULL, NULL, TRUE);
  }
}
?>

Si on regarde attentivement ce code, on voit en dur dedans:

<?php
$GLOBALS['base_url']
?>

Si on regarde la fonction url(), on verra quelque chose du genre:

<?php
$base = ($absolute ? $base_url . '/' : base_path());
?>

Je n'ai absolument aucune idée de pourquoi l'équipe Drupal à décidé de faire ainsi. Faire passer toutes les URL de fichier sans distinction par des URL absolues. Du coup, ça me bloque pas mal, puisqu'on utilise des sites derrière des proxy HTTP, on se retrouve avec des sites qui renvoient des URL absolues avec comme domaine les DNS internes de l'entreprise..

Et là, on est bien coincé, parce qu'on a pas envie d'aller patcher le coeur de Drupal sur les installations qu'on fait, pour plusieurs raisons, et surtout pour la maintenabilité.

Répondre

Le contenu de ce champ ne sera pas montré publiquement.
  • Les adresses de pages web et de messagerie électronique sont transformées en liens automatiquement.
  • Allowed HTML tags: <a> <em> <strong> <cite> <code> <ul> <ol> <li> <dl> <dt> <dd>
  • Les lignes et les paragraphes vont à la ligne automatiquement.
  • Use [fn]...[/fn] (or <fn>...</fn>) to insert automatically numbered footnotes.
N
S
E
b
w
v
Enter the code without spaces and pay attention to upper/lower case.