Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www/skrypt] Ocena skryptu komentarzy
Forum PHP.pl > Inne > Oceny
Stron: 1, 2
materkamil
Poprawiłem - teraz sprawdzam te z "x" ale znaki polskie dalej mi nie działają
redeemer
Nie tak się zabezpiecza skrypty przed XSS, nie wspomiając już o SQL injection. To co zrobiłeś to pójście po najmniejszej linii oporu - uniemożliwić użytkownikom wprowadzania niektórych znaków.

Widzę też przegladarka='$browsera'. Jeżeli to useragent to filtrujesz to w ogóle?
materkamil
Cytat
SQL injection


Na SQL Injection to jest w ogóle nie podatne! Nie ma znaków sql to nie ma ataku!

Cytat
Widzę też przegladarka='$browsera'. Jeżeli to useragent to filtrujesz to też?


Userzy nie wiedzą że ich informacje takie jak IP i przeglądarka są pobierane więc nie jest to filtrowane
!*!
Cytat(materkamil @ 4.05.2012, 12:00:11 ) *
Na SQL Injection to jest w ogóle nie podatne! Nie ma znaków sql to nie ma ataku!
Userzy nie wiedzą że ich informacje takie jak IP i przeglądarka są pobierane więc nie jest to filtrowane


Teraz wystarczy wstrzyknąć swój kod jako user agent i skasować całą bazę. I nie wiem co robisz źle, u mnie kod z wyrażeniem działa poprawnie.
r4xz
Cytat(materkamil @ 4.05.2012, 12:00:11 ) *
Userzy nie wiedzą że ich informacje takie jak IP i przeglądarka są pobierane więc nie jest to filtrowane


Sąsiedzi nie wiedzą że nie ma mnie w domu, więc po co zamykać drzwi?
materkamil
Sąsiedzi widzą otwarte drzwi a otwartej zmiennej nie widzą
redeemer
@materkamil: http://pl.wikipedia.org/wiki/Security_through_obscurity - polecam lekturę
!*!
Cytat(materkamil @ 4.05.2012, 12:05:51 ) *
Sąsiedzi widzą otwarte drzwi a otwartej zmiennej nie widzą


Skoro masz ją w kodzie który Nam pokazałeś, to co za problem wysłać własny nagłówek... Mówiłem Ci już, mogę wysłać do Twojego skryptu wszytko co tylko zechce i jeszcze więcej, a Ty się nadal upierasz że jest bezpieczny.

Nie ma sensu ciągnąć tematu dalej, wszytko zostało wyjaśnione chociażby tu Temat: WWW Strona o bezpieczenstwie komputerowym
materkamil
Dlatego zaraz jak to powiedziałeś dodałem filtrowanie nagłówka. A teraz jeszcze powiedz proszę jak zrobić aby skrypt dodawał do bazy tylko dane przesłane z formularza z mojej strony? Bo nie mam pomysłu kompletnie
usb2.0
http://php.net/manual/pl/book.mysql.php
!*!
Cytat(materkamil @ 4.05.2012, 13:01:24 ) *
Dlatego zaraz jak to powiedziałeś dodałem filtrowanie nagłówka. A teraz jeszcze powiedz proszę jak zrobić aby skrypt dodawał do bazy tylko dane przesłane z formularza z mojej strony? Bo nie mam pomysłu kompletnie


Filtruj dane. Czego w tym nie rozumiesz? PHP dostarcza Ci masę funkcji do tego, a google jeszcze więcej informacji. Zrezygnowałbym z mysql_* i wykorzystał PDO z bindowaniem danych, wtedy pozbywasz się ataku SQL In. zamieniaj znaki html na encje, a w loginie dopuszczaj tylko a-z0-9 i wykorzystaj to wyrażenie jakie zostało podane.
ciekawskiii
Az musialem sie zalogowac z gownianego telefonu zeby skomentowac bo czytalem jego 2 tematy w dziale ocena. I musze sie odezwac bo zal. Ja zajmuje sie amatorsko php od 4 lat i nigdy bym nie powiedzial ze sie na tym znam a Ty materkamil ile uczysz sie php? Ile masz lat? Sory za pytanie o wiek ale zachowujesz sie jak 10 letnie dziecko! Nie masz o podstawach pojecia! Nie wiesz jak filtrowac formularz i Ty chcesz uczyc innych? Gowno wiesz chlopaku! Zamknijcie ten temat bo to parodia! Byly o wiele lepsze projejty i tematy byly zamykane a tu jakiegos gowniarza temat jest ciagniety i ciagniety! Moderzy obudzcíe sie!
mls
Cytat(ciekawskiii @ 4.05.2012, 20:40:10 ) *
Ile masz lat? Sory za pytanie o wiek ale zachowujesz sie jak 10 letnie dziecko!


Sądząc po głosie - jeszcze mu daleko do mutacji, czyli wiek <12 lat, tak "na oko"...
ShadowD
Tak jak x osób powiedziało, PDO i bindy zmiennych i masz sporo z głowy, co do pól login/hasło ogranicz do [a-z0-9] ew. jakieś wybrane znaki, pomiń na pewno "<" i ">".

Ps. Nie komentując tego masowego spamu który ktoś Ci zagwarantował i nie da się praktycznie ogarnąć Twojej pracy.

Jeśli chodzi o temat, do oceny powinny być dawane jakieś projekty które da się oceniać, coś co ma jakiś rozmach, a nie prosty kod i to z błędami - toż to się nadaje do przedszkola i zatytułować "pomocy w naprawie filtrowania", bo jak tak dalej pójdzie każdy kto napisze jakikolwiek skrypt newsy/komentarze/logowanie będzie tutaj przychodził i dodawał swoje trzy grosze, jeszcze by było coś w tym specjalnego, jakieś multi-logowanie, super filtry anty botom/spamerom, coś co jest "modułem" i nadaje się do łatwej implementacji w innych projektach innych userów, ale "takie coś" - no proszę, toż to ktoś ogarnięty napisze kilka razy lepiej w 10 minut!
Daiquiri
"Służy ono (forum 'oceny' - Daiqurii) wyłącznie publikowaniu informacji o gotowych lub częściowo gotowych rozwiązaniach, co daje innym możliwość oceniania naszych osiągnięć." - Tematyka i zasady panujące na forum OCENY.

Niestety ten temat to pasmo niekończących się uwag na temat tego, że skrypt nie nadaje się do publikacji w tym dziale. Natomiast Twoja postawa Materkamil jest dla mnie kompletnie niezrozumiała. Prosisz o ocenę, potem wykłócasz się o każdą (mniej lub bardziej) negatywną uwagę. Temat zostaje zamknięty, ponieważ nie widzę sensu jego dalszego rozwijania w dziale ocen. Proponuję wrzucić swój skrypt do "Przedszkola" i tam kontynuować wymianę uwag dotyczącą jego zabezpieczeń.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.