[www] japierdziele.pl Opcje

[aiel]

Witam,

to moja pierwsza strona , sporo się przy jej tworzeniu nauczyłem.

http://www.japierdziele.pl

Użytkownicy dodają na niej swoje historie z życia pod ocene i komentarz publiki.
Bardzo prosze o konstruktywną krytyke

[r4xz]

[-] słabe wykonanie graficzne
[-] teksty z błędami ortograficznymi ("nie powiny też zawierać dużo ''byków''.")

podatności na różnych użyszkodników, którzy lubią pogrzebać w kodzie wolałem nie sprawdzać (sam formularz kontaktowy odstrasza (IMG:style_emoticons/default/worriedsmiley.gif) ). krótko mówiąc, czeka cię jeszcze wiele nauki... (IMG:style_emoticons/default/rolleyes.gif)

Ten post edytował r4xz 16.03.2010, 20:19:33

[aiel]

BARDZO dziękuje za każdą krytyke jest bardzo pomocna!

[-] słabe wykonanie graficzne
[-] teksty z błędami ortograficznymi ("nie powiny też zawierać dużo ''byków''.")

podatności na różnych użyszkodników, którzy lubią pogrzebać w kodzie wolałem nie sprawdzać (sam formularz kontaktowy odstrasza (IMG:style_emoticons/default/worriedsmiley.gif) ). krótko mówiąc, czeka cię jeszcze wiele nauki... (IMG:style_emoticons/default/rolleyes.gif)

Chciałem osiągnąc bardzo minimalistyczny wygląd , ale masz racje nie przyłożyłem się do layoutu jak powinienem.

Do tego podatność na SQL Injection na stronie z komentarzami. Mały przykład:

http://www.japierdziele.pl/index.php?id=%2...201%20IS%20NULL
http://www.japierdziele.pl/index.php?id=%2...IS%20NOT%20NULL

Dzieki za cynk naprawiłem to takim kodem mam nadzieje ze wystarczy :

[PHP] pobierz, plaintext 
if(is_numeric($_GET[id])){
// wyswietlanie komentarzy
 
 
 
}else {
 $ip=$_SERVER['REMOTE_ADDR'];
 echo "Wszelkie próby SQL injection są monitorowane <br>
  Skrypt zapisał twoje IP : $ip , do bazy";
  $sql = "INSERT INTO `ajpi` VALUES ('$ip', '$_SESSION[name]', '$_GET')";
  mysql_query($sql); 
  die;
}
[PHP] pobierz, plaintext 

Wystarczy wyłączyć obsługę ciasteczek w przeglądarce i możemy głosować bez końca. Miłego votowania.

Gdy dodawałem ten skrypt wiedziałem że to będzie możliwe , ale pomyślałem że tak bedzie najlepiej powód : nieobciązanie servera zapytaniami

dodawanie bez rejestracji i captcha ... prosisz się o spambota.
Przy rejestracji nie pisze ile co ma mieć znaków ...
strona podatna na xss

Captcha dodam jak tylko znajde jakąś minimalistyczna wersje
Z ilością znaków poprawiłem dzięki
A co do ataków xss musze się jeszcze o nich poduczyć i napewno zabespiecze strone