[extra-c]

Ursprungligen postat av StefanW

Jag förstår inte riktigt poängen med siten. Samt har du tänkt på regler på innehåll? Jag hittade inga. Du ansvarar själv för innehållet på din webbplats, och beroende på webbhotell (har inte kollat vilket du kört), men de flesta tillåter inte tex porr, barnborr osv.

Har du någon validering på filerna som laddas upp? Tex att du måste godkänna allting? I så fall kan det blir mycket jobb om siten blir stor (vilket jag inte ser då jag inte ser syftet i den).

Designmässigt är den väl Ok, det finns inte så mycket i design att kommentera.

Koden skulle kunna göras mycket bättre. Väldigt ostrukturerad, mycket inline-css och den validerar inte alls http://validator.w3.org/check?uri=re...Inline&group=0

(169 errors och 6 varningar)

Tack för ditt svar! Jag kan hålla med om att koden kan förbättras och det kommer jag att göra framöver.

Idén är att man ska kunna publicera saker, det är som en sorts "klotterplank". Jag vet att det inte är någon vidare idé. Men jag hade tänkt att utveckla det , alltså själva idén och tillägga flera funktioner.

Regler och användarvillkor hade jag glömt, hade den känslan att något saknades. Ska lägga till det imorgon eller under veckan.

Allt som skickas in läggs upp automatiskt, sedan det som känns olämpligt kan man radera via databasen. (Admin panel på gång).

Ursprungligen postat av StefanW

Tillägg: Ingen validering på filtyp. Lyckades ladda upp en php-fil. (ofarlig)

En till sak; visar du alla kommentarer på ett inlägg? Om ett inlägg får 1000 kommentarer blir det en himla massa att scrolla.

Validering på filtyp bör fungera, testade själv att ladda upp .PHP fil eller andra filer som inte var någon bild och då får man fram "Endast filer av typerna .jpg, .jpeg, .png, .gif är tillåtna att laddas upp.".

Bra att du nämnde det! Ska lägga till under veckan ett script som gör att den visar maximalt 6 kommentarer och sedan får man trycka på "Visa mer".

[JesperA]

Ursprungligen postat av extra-c

Validering på filtyp bör fungera, testade själv att ladda upp .PHP fil eller andra filer som inte var någon bild och då får man fram "Endast filer av typerna .jpg, .jpeg, .png, .gif är tillåtna att laddas upp.".

Nja alltså det går ju fortfarande att ladda upp PHP filer om man döper om den till .jpg osv.
Att kolla om en fil verkligen är en bildfil är lite krångligare än att bara kolla filändelsen dom du gör nu.

[extra-c]

Ursprungligen postat av JesperA

Nja alltså det går ju fortfarande att ladda upp PHP filer om man döper om den till .jpg osv.
Att kolla om en fil verkligen är en bildfil är lite krångligare än att bara kolla filändelsen dom du gör nu.

Jaha, fast döper man om filen till .jpg och laddar upp den så kan inte PHP göra någon funktion? Eller? Det jag menar är att om du har <?php echo "lol"; ?> i filen och den laddas upp som .PHP fil så visar den "lol" men om du laddar upp den som .jpg fil så kommer det antingen stå att bilden kan inte visas för att den är skadad eller visa hela koden: <?php echo "lol"; ?>.

Eller tänker jag fel?

[JesperA]

Ursprungligen postat av extra-c

Jaha, fast döper man om filen till .jpg och laddar upp den så kan inte PHP göra någon funktion? Eller? Det jag menar är att om du har <?php echo "lol"; ?> i filen och den laddas upp som .PHP fil så visar den "lol" men om du laddar upp den som .jpg fil så kommer det antingen stå att bilden kan inte visas för att den är skadad eller visa hela koden: <?php echo "lol"; ?>.

Eller tänker jag fel?

Du kommer ju få brutna bilder åtminstonde, http://www.rednation.se/sv/index.php?p=public&m=10 (japp du tog bort den medans jag skrev)

Du kan göra mycket för att kolla lite extra, kolla bredden/höjden på bilden med getimagesize(), mime typen, efter du flyttat filen till din upload folder kan du kolla en extra gång om filen faktiskt finns där, ibland kan move_uploaded_file() vara true fastän den failat med filen, det finns flera andra sätt som du kan dubbelkolla bilduppladdningen, så när något av dessa failar, skicka inget alls till databasen.

Det finns många andra bättre sätt att verkligen garantera att det fungerade, bara att googla lite så hittar du alltid något.

[extra-c]

Ursprungligen postat av JesperA

Du kommer ju få brutna bilder åtminstonde, http://www.rednation.se/sv/index.php?p=public&m=10 (japp du tog bort den medans jag skrev)

Du kan göra mycket för att kolla lite extra, kolla bredden/höjden på bilden med getimagesize(), mime typen, efter du flyttat filen till din upload folder kan du kolla en extra gång om filen faktiskt finns där, ibland kan move_uploaded_file() vara true fastän den failat med filen, det finns flera andra sätt som du kan dubbelkolla bilduppladdningen, så när något av dessa failar, skicka inget alls till databasen.

Det finns många andra bättre sätt att verkligen garantera att det fungerade, bara att googla lite så hittar du alltid något.

Heheh ja det var jag som tog bort denna ^^. Jo okej, tack för hjälpen. Ska kolla runt på detta och utveckla det ännu mer!