Discussion :

[laurentSc]

Bonsoir,

dans le code ci-dessous, on ne passe pas dans le foreach qui affiche la valeur retournée par la requête SQL (quelconque). Pourquoi ?

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
if (!empty($sql)) {
	try{
		$bdd = new MyPdo();
	}catch(PDOException $e){
		echo "argggggggggggggg".$e->getMessage();
	}
	echo "<table width='100%' border='1' style='text-align:center'><tr>";
	$first = true;
	var_dump($bdd); //on passe bien là
	foreach ($bdd->query($sql, PDO::FETCH_OBJ) as $ligne) {
		var_dump($ligne); //on ne passe pas là
		if ($first) {
			echo "<tr>";
			foreach ($ligne as $key => $value) {
				echo '<th>' . $key . '</th>';
			}
			echo "</tr>";
			$first = FALSE;
		}
...

[ABCIWEB]

Bonjour,

Et pourquoi tu ne mets pas la requête à l'intérieur du try{} ? C'est bien pour s'en servir qu'on a déclaré le mode exception dans l'initialisation de la connexion. Cela te donnerais les messages d'erreur appropriés.

Il me semble aussi qu'on a déjà déclaré également le mode PDO::FETCH_OBJ dans l'initialisation de la connexion, donc encore une fois (et je te l'ai déjà dit) IL N'Y A PAS BESOIN DE LE REDECLARER PAR LA SUITE.

Pour le reste je comprends pas trop l'histoire du deuxième foreach.

[sabotage]

Pourquoi ?

Parce que ta requête ne retourne rien, soit elle produit une erreur, soit elle ne produit aucun résultat.

[Invité]

Bonjour Laurent,
tu n'as pas l'impression d'écrire le code en dépit du bon sens ?
C'est quand même toujours plus ou moins écrit n'importe comment, non ?
Et tu refais toujours plus ou moins les mêmes erreurs de débutant...

On est encore quelques uns à avoir encore la bonne volonté de t'aider.
Tu vas finir par épuiser tout notre quota de patience...

Même ABCIWEB est en train de craquer...

[laurentSc]

Pas la peine de t'énerver ABCIWEB ; en fait, ce n'est pas la même application : ici, c'est un phpMyAdmin-like (bon, pas vraiment), en gros un script qui me permet d'exécuter des requêtes SQL et d'en afficher le retour s'il y en a. (J'avais fait ça à une époque car chez Free, je trouvais phpMyadmin hyper lent donc je préférais m'en passer et en gros, je n'y ai recours que pour faire des imports/exports, et en ayant pris l'habitude, je m'en sers chez tous les hébergeurs).
Mais du coup, j'ai aligné le connect.php sur l'autre, et du coup, j'ai enlevé le PDO::FETCH_OBJ ; d'autre part, j'ai aussi intégré la requête dans le try.

Sabotage, je peux tester cela avec n'importe quelle requête et même celles qui doivent retourner quelque chose ne retourne rien ici.

En fait, même avec echo "<br/>argggggggggggggg ".$e->getMessage(); dans le catch, ça affiche :

argggggggggggggg SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '
show tables
' at line 1

si je teste avec show tables.

Bon, Jérôme, ce code est assez ancien, même si je l'ai converti de _mySQL à PDO il y a peut-être 2 ans, mais comme jusque là, il marchait, je ne le modifiais pas...et les erreurs de débutant ici, je les vois pas...

[Invité]

La première "erreur de débutant" est que tu ne sais pas encore effectuer un débogage de base.

Ensuite, c'est de ne pas appliquer les connaissances que tu as apprises, ou les conseils qui t'ont été donnés.

Que le code soit ancien ou pas, tu es censé utiliser tes connaissances nouvelles.

[Bovino]

y a que moi qui en connais les URLs

No more comments...

[laurentSc]

Bon OK, il serait plus sûr de proposer une liste d'instructions dans une liste déroulante et de laisser libre le reste, si je mets pas le DROP dans la liste, sauf que je m'en sers aussi, donc comment faire ? Et d'autre part, qu'est-ce que je risque ?

[Invité]

qu'est-ce que je risque ?

Que quelqu'un, malgré tout, trouve l'URL du fichier (non protégé !) !
Et sachant que tu utilises sûrement le même type d'URL sur chacun de tes sites, TOUS TES SITES seront vulnérables !
(du genre : www.site1.com/admin/meme-pas-peur.php - www.site2.com/admin/meme-pas-peur.php - ... !)

..., donc comment faire ?

Lire/étudier les conseils déjà donnés.

...faire un formulaire d'authentification avec login et mot de passe. C'est pas du temps perdu car tu pourras t'en servir pour tous les espaces administrateur.

+ voir aussi :

• Qu'est-ce qu'un fichier .htaccess ?
• Comment mettre en place une authentification simple des utilisateurs ?

[ABCIWEB]

Tu peux trouver ici un exemple de formulaire complet qui permet en plus un hash du mot de passe pour éviter qu'il transite en clair dans les tuyaux.

L'autre solution donnée en dernier exemple par jreaux62 (authentification via http et htaccess) est plus rapide à mettre en place, mais elle est moins souple d'utilisation, moins transposable d'un site à l'autre et elle n'est pas personnalisable. Mais pour aller vite c'est le minimum vital.

Enfin bon il faudrait que tu t'intéresse sans plus tarder à la protection d'un espace administrateur (appelé aussi "espace membres" si tu veux plus de réponses dans google). Normalement cela se fait naturellement avant la création d'un espace administrateur, pas après, c'est bizarre que tu n'aies pas compris cette priorité. Tu te serais aperçu qu'on utilise jamais la méthode que tu utilises qui est vraiment trop puérile (en fait elle n'est même jamais mentionnée).

[Bovino]

J'imagine que la notion d'utilisateur et de privilèges pour une base de données t'es inconnue...