Header php true or false ?

[KOogar]

Bonjour,

Je dois rediriger
---https://monsite.fr/code_s.php?id=1189
vers
---https://monsite.fr/code/php/applications/1189_calculette-basique-en-php-et-html

Vu que j'ai pas loin de 1k d'urls a traiter, je veux être certain de bien utiliser les headers

voici le code, est-ce bon ?

if (isset($_GET['id'])) {
    
    $id_esc = $_GET['id'];
    $q_s = $conn->query("SELECT * FROM $table_mysql WHERE idx = '$id_esc' ");

    $r_s = mysqli_fetch_array($q_s);

    // La requête est vide
     if ($q_s->num_rows == 0) {
        header('Status: 301 Moved Permanently', FALSE, 301);
        header('Location: ' . $url_site);
        exit;
    }
    else {
    // C'est correct, on fait la redirection sur la page 
    $url = url_code_source($r_s['idx'], $r_s['cat'],$r_s['titre']);
    $url = $url_site.$url;
    header('Location: ' . $url, TRUE, 301);

    }
  }
 else
 {
 // Pas de $_GET['id'] identifié
 header('Location: ' . $url_site,  FALSE, 301);

 }

 

[spout]

Attention à la faille d'injection SQL :

?id=' OR 1=1; DROP TABLE users;--

Sinon ça à l'air bon, suffit de tester les entêtes avec un outil.

 

[spout]

Et le true ou false c'est pr remplacer ou pas si t'as déjà envoyé un header similaire.

 

[colonies]

Ça ne serait pas plus simple comme ça ?

$page = "";
if (is_numeric($_GET['id'])) {
   $sql = "SELECT * FROM $table_mysql WHERE idx = " . (int) $_GET['id']
   $qr = $conn->query($sql);
   // page trouvée
   if ($qr->num_rows) {
      $row = mysqli_fetch_array($qr);
      $page = url_code_source($row['idx'], $row['cat'],$row['titre']);
   }
}

header('Location: ' . $url_site . $page,  TRUE, 301);
exit;

 

[KOogar]

@colonies
merci, c'est + simple et mieux codé
Manquait juste un ; à la fin de la requête