.: VIDÉOTHÈQUE V 0.2.1. :.

Commentaire de codefalse le 08/01/2008 22:49:36 administrateur CS

Le concept est bon, l'idée est agréable :) mais quelques remarques :


§index.php
# <?
#
# // On charge le main par l'intermédiaire de la page d'accueil
# print "
# <SCRIPT langage='Javascript'>
# window.location='main.php'
# </script>
# ";
# ?>

C'est horible ! :p Pourquoi ne pas faire une vraie page html, c'est plus vrai et plus beau

§main.php
# include ('pages/'.$_GET['page'].'.php');
C'est dangeureux ! Pas de vérification au niveau des entrées utilisateurs

Dit toi que TOUT ce qui provient de l'utilisateur DOIT ETRE ANALYSE !!!!!
Ne fait JAMAIS confiance à tout ce qui vient de l'extérieur (meme les valeurs _SERVER[] peuvent être modifiée !!)

On voit que ton code à subit des modifications et n'est pas tout frais, il est apparement fait pour un serveur en Php4 alors que c'est fini pour cette version (d'ailleur tu devrais bientot avoir des problèmes quant à ca ! :p)


§ about.php
#  print "<div id='about'>
# .:. Vidéothèque .:. V 0.2.0. <br />
# Logiciel de gestion des vidéos peronnelles.<br />
# Configuraion minimum : Mysql 3.23, Php 4.1<br />
# Développement par <a href='homedev.free.fr'>.: HOMEDEV :. </a><br />
# Développé par Vincent BUJON<br />
# --------------------------------------------------------------------<br />
# <br />
# Licence GPL - GNU <br />
# <br />
# </div>";

Idem que la premiere remarque

Bon bref j'ai pas tout regardé, mais pense à la sécurité ! Surtout si (est c'est tres probablement le cas ici !) tu fait tourner ce code sur ton pc, les problemes de sécurité présent risquent de te poser de sérieux problemes !

En tout cas courage pour la suite :)

Commentaire de xstyled le 21/01/2008 21:53:27

Salut,

J'ai regarder vite fais quelques lignes, l'idée est toujours sympa. Par contre pour ton systeme de login, je  ne pense pas que les gens puissent se logguer puisqu'il n'y a nul part un session_start .

<?php
session_start();
?>
dans main.php par exemple.
Bonne soirée ;)

Commentaire de xstyled le 22/01/2008 18:28:50

Yop,

je ne suis pas sur que l'on puisse etre identifié, vu les emplacements des session_start, pour l'occasion je vais l'essayer.