http://hackme.boo.pl

Proszę o ocenę mojej strony:
1) layout
2) treść (czyli głownie zadania)
3) bezpieczeństwo
4) Ogólne wrażenie

Na tło dal bym jakieś kreseczki bo przy ~1600/1000 monotonne jest strasznie, napis na logu zlewa się z nim.

Dodatkowo powinieneś ostawić jakieś testowe konto by nie trzeba było się rejestrować...

Ps Dwa błędy w html wypadało by poprawić...

Pierwsze wrażenie? "Coś pusto tu".


Schludny aż do przesady.


:

dodałem konto testowe i strona validuje się poprawnie

Login: tester
Hasło: test

"<form>" też da się css-ować!
zrób to, gdyż na Twej stronie są inputy ze standardowym wyglądem i nie wygląda to dobrze.
Dodatkowo gdyby w tym górnym panelu były powoli spadające literki zamiast statycznego obrazu byłoby zaje..fajnie.

I za dużo bieli - gdzieś musisz dodać tło - w background może być ciemniejsze, w reszcie np. jasny błękit

@timonie27 owszem da się, ale to nie ma sensu, bo każda przeglądarka i tak pokaże to inaczej, że już nie wspomnę o niepoddawaniu się na style inputa do pobierania plików. (Widziałem gdzieś w necie stronę, na której były pokazane różne inputy z różnych przeglądarek, chętnie bym Cię tam odesłał, ale niestety nie znam adresu [może ktoś zna?]).

http://www.456bereastreet.com/archive/2004..._form_controls/

Błękitny kolor nie pasował, więc na tło dodałem lekką szarość. Były chwilowe problemy z testerem bo ktoś zmienił hasło ale już jest wszystko w porządku. Można się logować:
Login: tester
Hasło : test

erix
Dzięki za link

Ogólne wrażenie:
1. Popracować nad kolorystyką:
2. Popracować nad typografią

Kod z początku wydawał się niczego sobie, choć wypadałoby choć jakies wcięcia stosować dla lepszej czytelności. Tak sobie przeglądałam i patrzę a tu taki kwiat:

[HTML] pobierz, plaintext 
<div id=text2>
<table id=news>
<tr><td id=newstytul>Witam</td></tr>
<tr><td id=newstresc><p>Witam  na hackme.boo.pl. Na razie występują tu tylko zadania związane z bezpieczeństwem stron www, ale w przyszłości zamierzam dodać także zadania matematyczne oraz związane z kryptografią.  A tymczasem zapraszam do rejestracji.</p>
</td></tr></table> 
<br>
</div>
[HTML] pobierz, plaintext 

Nie dość, że jednokolumnowa tabelka (nie wiadomo po co) to jeszcze w divie i w dodatku do zrobienia odstępu użyty znacznik <br>. A fuj...

Poza tym nigdzie nie zauważyłam choćby <h1> i nagle nie z gruszki ni z pietruszki w profilu wyskakuje <h4>

Takie coś:

[HTML] pobierz, plaintext 
Login: tester
<br>
Mail: cos@cos.pl <br>
 
Punkty: 5 <br>
Ranga: user<br>
[HTML] pobierz, plaintext 

To aż się prosi o listę <ul>

Na koniec taka uwaga. Namiętnie i z uporem maniaka pomijasz cudzysłowy w których powinny być wpisane wartości atrybutów znaczników, czasem tylko zamiast nich stosujesz apostrofy.

Hmmm Nie można zalogować się do nierozwiązanych zadań

Dzięki za rady. Postaram się to w najbliższym czasie poprawić.


Nie rozumiem dokładnie o co Ci chodzi. Mógłbyś to rozwinąć ?

Mogłabym

Otwieram listę zadań. Jest tylko jedna kategoria, WWW. Klikam na nią wyświetla się lista zadań. Klikam w zadanie pojawia się okno zachęcające do wpisania hasła. Wpisuję hasło i dowiaduję się, że jest nieprawidłowe i niniejszym nie mogę obejrzeć zadania.

Chyba źle zrozumiałaś po co jest tam to hasło.
Nad inputem do wpisania hasła jest link do zadania. Gdy rozwiążesz zadanie otrzymasz hasło, które musisz wpisać właśnie do tego inputa. Gdy to zrobisz otrzymasz punkty.

Wszystkie kwiatki poprawiłem

Przejrzyj wszystkie linki które wysyłasz na mejla. Bo ja dostałem raz coś takiego na początku linku:http://zadania.skate.boo.pl/

Przy okazji w tytule zjadło polską literę.

Nie wiem jak to możliwe że taki link dostałeś. Pokaż dokładnie jak wyglądała wiadomość.

Dzięki. Przenosiłem z innego serwera i dlatego wystąpił taki błąd.

To może i ja się wypowiem.

Wygląd:
Strona bez życia. Nic mnie w nim nie pociąga. Szarawe tło to bardzo kiepski pomysł. Amatorskie gradienty, za duże bordery. Bardzo kiepsko.
Logotyp nie jest klikalny. Wszędzie ten sam, mdły kolor. Brak wyróżnienia pozycji menu, w której aktualnie się znajdujemy.
Ogólnie: 3/10

Kod html:
Brak jakiejkolwiek tabulacji! Słowa kluczowe za mało mówiące o stronie, a opis jest genialny. Dlaczego nazwy class w divach nie wpisujesz w cudzysłowach? Następnym razem daj do oceny kod nad którym pracowałeś więcej niż godzinkę.
Ogólnie: 2/10.

Kod css:
Brak tabulacji!

[HTML] pobierz, plaintext 
font-family: "Trebuchet MS";
[HTML] pobierz, plaintext 

Używaj bardziej uniwersalnych czcionek, a jeżeli już musisz taką podać, to jako drugą w kolejności wpisz uniwersalną. Nie ujednolicasz sobie pisania kodu css. Zauważyłem, że czasami na początku dasz padding-i, a czasami font-y. Musisz zastosować hierarchię. Wtedy będzie Ci się łatwiej edytowało kod.

[HTML] pobierz, plaintext 
border-style: solid;
border-width: 2px;
border-color: #337de1;
[HTML] pobierz, plaintext 

Takie coś robi się za pomocą samego border-a.

Działanie:
Po zalogowaniu się muszę kliknąć "Dalej". Czy nie lepiej od razu odświeżyć stronę? Chciałem zobaczyć o co chodzi z tą stroną, ale w pierwszym zadaniu wyświetla brak pliku.

Przyznaję że nie potrafię robić layoutów. Nie mam daru do łączenia kolorów dlatego i tak jestem zadowolony z tego co stworzyłem.
Ale dzięki za krytykę. Na pewno da mi to coś do myślenia. Opisy, kod są jakie są, bo nie przykładałem się do nich. Bardziej myślałem nad kodem php. Ale w najbliższym czasie popracuję nad tym kodem html i css.



Zastanawiałem się nad tym odświeżaniem strony, ale wybrałem link "Dalej". Możliwe że to zmienię.

Przepraszam za to pierwsze zadanie. Wczoraj niechcący usunąłem jeden plik. Już jest wszystko w porządku.

Trebuchet MS nie jest uniwersalny wg. Ciebie? To jedna z najczęściej używanych czcionek dzisiaj. Ale fakt, że wers CSS powinien brzmieć:



Dodanie 'sans-serif' chyba wiadomo co spowoduje;)



Ciekawa teoria, ale chyba niewykonalna. Z tego względu, że pisząc kod CSS zwłaszcza pod windowsem testujesz, sprawdzasz dodajesz i odejmujesz wersy i patrzysz jaki jest efekt. Ciężko zachować tę samą kolejność, zresztą kod CSS jest tak prosty, że nie ma tam za bardzo co analizować

Gdzie ty żyjesz? Dwie magiczne litery na końcu nazwy fonta mówią wszystko. Sprawdziłem na ubuntu 8.10 i takiej czcionki nie ma, a paczki z windowsowskimi czcionkami jak na razie nie zainstalowałem.

Jak to niewykonalna? Ja jakoś sobie radzę i bardzo ułatwia mi to pracę. Mam taki system (od góry): definiowanie bloku, font-y, rozmiary, background-y, border-y, padding-i, margin-y. Te 3 ostatnie kiedyś mieszałem, ale ostatnio trzymam się zasady.

Tak niestety jest, Trebuchet MS jest wszędzie, nie tylko na stronach, ale b. często w reklamach, szyldach itp.
Spoko, też mam Ubuntu i wiem, że nie ma Ale mam też windę i nie znam żadnej czcionki bezszeryfowej, która jest i pod Linuksa i pod Windowsa dostępna jako standard. Po prostu dodajesz sans-serif na końcu i użytkownicy windy mają np. Trebuchet, Verdane, czy Arial, a linuksowcy zobaczą np. DejaVu SansSerif



No to tak trzymaj. Jednak jesteś jedyną osobą jaką znam, która tak postępuje

pzdr;

A co Ci do tych tabulacji?! Im mniej białych znaków tym lżejszy plik. Przeglądarce nie robią one żadnej różnicy(!), a jeśli sam chcesz analizować źródło to możesz sobie skorzystać z Firebuga i będziesz miał wcięcia.



Osobiście nie hierarchizuje styli w {} i nie wcale nie mam trudniej.

Wybacz, ale chyba nigdy nie napisałeś bardziej złożonego kodu. Pokaż mi porządną stronę bez tabulacji! Pokaż mi złożoną klasę w php bez tabulacji! Pisz 3 warunki i każdy kolejny w poprzednim. Ciekawe ile będziesz główkował co do czego. Zapraszam na PW, żeby nie robić off-topa.

Kto jak lubi. Mi to ułatwia i przestrzegam wytyczonej hierarchii.

Hola kolego, nie mówimy o kodzie php, o html i css. [koniec oftopa /]

Hmmm, zadania mają złą kolejność np zadanie 9 (Bez hasła) za pierwszym razem poszło a nad tymi za 10pkt siedziałem i nic... ;p

To zależy dla kogo. Dla jednego zadanie związane z JS jest banalne dla drugiego nie. Zadania w których można podejrzeć źródło są mniej punktowane. Czyli JS i proste zadania w których właściwie wystarczy spojrzeć w źródło. Zaś te w których nie da się podejrzeć źródłą skryptu są więcej punktowane, gdyz musisz poprostu domyślić się jak dany skrypt jest zbudowany.

Dlatego powinieneś zacząć używac programó, któe są pomocne przy dobieraniu kolorystyki (sporo tego w sieci jest).



No to ja bym się wstydziła przyznać do tego, że dałam do oceny kod do którego się nie przyłożyłam

Bardzo często przychodzi mi mail "Mail delivery failed: returning message to sender", przez to że użytkownicy podają nie istniejące adresy email. I teraz pytanie do Was. Czy jest możliwość sprawdzenia przez skrypt php, czy dany adres email istnieje ?

Nie zgodzę sie z tym. Poza tym popatrz na fora (np na TO) inputy są poważnie przetworzone (nie tylko kolory, obramowanie, ale i wielkość i krój czcionki). Wyobraź Sobie jak forum php.pl wyglądałoby ze standardowymi inputami. A w wielu przeglądarkach sprawdzałem i wyglądają tak samo.


Nie. A przynajmniej nic o takiej techologi nie słyszałem (po samej nazwie nie poznaszbo każdy może mieć konto na własnym serwerze).
Możesz jedynie sprawdzać podstawowe sprawy jak:
-czy w adresie znajduje się dokładnie jedna małpa
-czy po małpie jest co najmniej jedna kropka
itp
Zazwyczaj tak się to robi.

pelna sciezka do twoich zadanek:



uwazaj troche co piszesz...

generalnie duzo mozna sie dowiedziec o tym co tam trzymasz u siebie:

np. takie zapytanie:



wynik:



// edit

generalnie nie powinno sie ujawniac takich informacji

Strona merytoryczna: ogólnie lubie hackme... twoje jest troche proste, zrobilem all zadania prócz ostatniego jak narazie, design bardzo lekki

Tak już mam zrobione.




Tylko jak zabezpieczyć się przed tym ? Skoro zadanie polega na SQL injection ?

co do ścieżek to ustaw jakis error_handler

co do mysqla to moze jakies wyrażonka regularne?

A w jaki sposób zdobyłeś tą ścieżkę ?

w ktoryms z zadań (nie tylko w jednym) wsytarczylo podawac w zmiennych jakies bledne dane, naprzyklad bylo takie zadanie z bledem LFI, tam podalem jakies dane, bo myslalem za pierwszym razem ze to SQL Injection i wyskoczyl blad...

Mógłbyś oznaczać jakoś zrobione zadania. Niektóre chce na przykład zostawi na potem i teraz nie wiem które rozwiązałem a które nie

niepoprawne pisze się razem, nie oddzielnie

Dzięki, poprawione, choć nie jestem pewien czy w każdym pliku poprawiłem.


Miałem to w planach wcześniej. Niedługo to dodam.

Super by było, bo jak znajdę trochę czasu, chętnie powrócę do gry, żeby mieć wyższe miejsce w rankingu.

jak jest link do zadania , np:

Zdobądź hasło na stronę.
Oto link od zadnia :
http://zadania.hackme.boo.pl/zadanie2/

to mógł by się otwierać w nowej karcie...

Poprawione...

Athlan: A odnośnie tego sprawdzania to jak wejdziesz w zadania to jeżeli jest zrobione pokaże się napis:

Ale muszę wchodzić w każdy task. To tak jakbyś nie miał na forum oznaczonych tematów nieprzeczytanych i musiał klikać w każdy, żeby sprawdzić, czy doszedł jakiś post.

Ja nie powiedziałem, że tak zostanie.Tylko to może ci posłużyć tymczasowo.

wchodząc w podglad usera widzę jego email
No jak tak traktujesz swoich userow, ze upubliczniasz ich adresy email, to ja dziekuje za taki serwis...

no chyba ze user jawnie wyraził zgodę na otrzymywanie spamu przez spamboty, ktore w ten sposób zdobedą jego email

Moim skromnym zdaniem jest tak:
- Witryna "daje frajde" z hackowania bazy danych itp - sprawdzenia umiejętności (...), ale sam autor nie umie się zabezpieczyć przed takimi atakami.

Design: 4/10 - Białe standardowe tło jest brzydkie, spróbuj inny kolor, może jakieś lekkie paski? eee... nie nie pasuje.

Pozdrawiam, WebNuLL

Właśnie jeszcze o tym miałem napisać, ale wyleciało mi z głowy. Nie życzę sobie podawania mojego adresu email publicznie. Nie akceptowałem żadnego regulaminu i nie miałem do przeczytania żadnej polityki prywatności.

Proszę o usunięcie z mojego profilu adresu emailowego. (bo spamboty przyjdą, zabiorą i dostane spama ;p)

Za ten email przepraszam, jestem właśnie podczas dodawania możliwości ukrycia emaila.

To nie jest jakis malutki bug. Zanim dodasz te mozliwosc, najzwyklej w swiecie go do tej pory nie wyswietlaj.
Po dodaniu tej mozliwosci domyslnie ustaw ze ludziom sie nie wyswietla. Nie sądze by ktos wogole chcial go ustawiac by wyswietlal

Dokładnie tak zrobiłem zanim przeczytałem twój post

w zadaniach kod html jest zły:P
przykład:

[HTML] pobierz, plaintext 
<head>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
</head><html>
<form action=login.php method=post>
Login : <input type=text name=login> <br>
Hasło : <input type=password name=haslo> <br>
<input type=submit value=Loguj>
</form>
Zły login lub hasło
</html>
[HTML] pobierz, plaintext