Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Mój skrypt uploadu, proszę o opinie
cisu
post
Post #1





Grupa: Zarejestrowani
Postów: 157
Pomógł: 1
Dołączył: 15.03.2008
Skąd: Mirostowice Dolne

Ostrzeżenie: (0%)
-----


Witam,

dzisiaj chcę oddać Wam do testowania mój skrypt uploadu i udostępiniania plików.

Główne funkcje skryptu:
=> upload plików
=> usuwanie plików
=> rejestracja, logowanie, reset hasła, zmiana hasła, zmiana e-mail
=> dla zalogowanych wgląd do swoich plików (możliwość sortowania), możliwość zakładania hasła na każdy plik
=> przed pobraniem cudzego pliku trzeba przepisach captcha (wykorzystałem tu reCAPTCHA)

Z racji, że to wersja beta, maksymalna wielkość pojedyńczego pliku to 100kb.

Jeśli chodzi o wygląd, grafikę itd. to jest wersja robocza i będzie zmieniona, wraz z logiem.

Do wysyłania wiadomości wykorzystałem PHPMailer.

Chodzi mi o działanie skryptu, czy nie ma błędów itd.

Nie ma panelu administratora, ale będzie. Wszystkie pliki będą usuwane przeze mnie co jakiś czas.

Adres skryptu to:
Kod
http://upload.icisu.net/

Proszę o zgłaszanie jak coś nie działa oraz różnych sugestii.
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
Rid
post
Post #2





Grupa: Zarejestrowani
Postów: 715
Pomógł: 47
Dołączył: 5.12.2010

Ostrzeżenie: (0%)
-----


A jak tam ze sprawdzaniem heurastyki??Można każdy plik wrzucić,a co jeśli ktoś wrzuci wirusa?(IMG:style_emoticons/default/questionmark.gif)
Go to the top of the page
+Quote Post
mortus
post
Post #3





Grupa: Zarejestrowani
Postów: 2 178
Pomógł: 596
Dołączył: 25.09.2009
Skąd: Piwniczna-Zdrój

Ostrzeżenie: (0%)
-----


Cytat(Rid @ 10.09.2011, 12:33:14 ) *
A jak tam ze sprawdzaniem heurastyki??Można każdy plik wrzucić,a co jeśli ktoś wrzuci wirusa?(IMG:style_emoticons/default/questionmark.gif)

Nie heurastyki, a heurystyki. Poza tym nie można sprawdzać heurystyki, bo heurystyka to właśnie sprawdzanie metodami przybliżonymi, czyli takimi, które nie dają stuprocentowych rozwiązań. Pytanie zatem powinno brzmieć "A co z heurystycznym badaniem uploadowanego pliku w celu wykrycia potencjalnego zagrożenia?". Równie dobrze można zapytać, czy w ogóle został zaimplementowany jakiś mechanizm sprawdzania, czy plik nie jest przypadkiem zainfekowany.

Co do skryptu:
- uploader nie ma przycisku (krzyżyka) zamknij, a tylko anuluj, którego naciśnięcie wyświetla komunikat (a co mam zrobić, jak już skończę uploadować pliki?)
- komunikaty pojawiają się na środku (np. podczas kopiowania linków do pliku, czy usuwania plików), co mnie osobiście irytuje,
- wszystko jest trochę za małe, menu jest mało intuicyjne,
- może warto by było zaimplementować możliwość podglądu treści pliku.

Konto testowe nie powinno posiadać możliwości zmiany danych.
Go to the top of the page
+Quote Post
cisu
post
Post #4





Grupa: Zarejestrowani
Postów: 157
Pomógł: 1
Dołączył: 15.03.2008
Skąd: Mirostowice Dolne

Ostrzeżenie: (0%)
-----


Cytat(mortus @ 10.09.2011, 12:49:52 ) *
- uploader nie ma przycisku (krzyżyka) zamknij, a tylko anuluj, którego naciśnięcie wyświetla komunikat (a co mam zrobić, jak już skończę uploadować pliki?)
Komunikat jest zabezpieczeniem, żeby nie przerwać przypadkowo wysyłania plików, napisz Anuluj mogę zmienić na Zamknij, to nie jest żadnej problem
Cytat(mortus @ 10.09.2011, 12:49:52 ) *
- komunikaty pojawiają się na środku (np. podczas kopiowania linków do pliku, czy usuwania plików), co mnie osobiście irytuje,
Tak jest chyba najbardziej przejrzyście, gdzie niby mają się pokazywać?
Cytat(mortus @ 10.09.2011, 12:49:52 ) *
- może warto by było zaimplementować możliwość podglądu treści pliku.
Nad tym pracuję, tzn. będzie możliwość podglądu plików tekstowych, obrazów, może pdf
Cytat(mortus @ 10.09.2011, 12:49:52 ) *
Konto testowe nie powinno posiadać możliwości zmiany danych.
To jest zwykłe konto, zrobiłem je, bo z doświadczenia wiem, że ludzie nie chcą się rejestrować.



Cytat(toaspzoo @ 10.09.2011, 13:00:56 ) *
Widzę, że jest szyfrowanie hasła nawet ;P

Hasło jest hashowane, a gdzie to widzisz?
Go to the top of the page
+Quote Post

Posty w temacie
- cisu   Mój skrypt uploadu   21.05.2011, 16:11:18
- - 1010   100kb? Co tu w takim razie oceniać? Upload to tylk...   21.05.2011, 19:22:43
- - Daimos   Przy małych plikach może to przeoczyłem, ale nie m...   21.05.2011, 19:27:05
- - cisu   Co do 100kb to jak napisałem, jest to wersja testo...   21.05.2011, 19:35:27
|- - r4xz   Cytat(cisu @ 21.05.2011, 20:35:27 ) N...   21.05.2011, 20:56:35
|- - cisu   Cytat(toaspzoo @ 21.05.2011, 21:51:05...   21.05.2011, 23:01:10
- - toaspzoo   Założyłem konto Fajne to. Mam pytanie, skrypt dz...   21.05.2011, 20:51:05
- - fifi209   A ciężko założyć konto testowe? Jak założysz, będę...   21.05.2011, 23:16:16
- - konole   Czekaj, to u ciebie uploaduje się plik w jakim cel...   22.05.2011, 14:30:44
|- - cisu   Cytat(konole @ 22.05.2011, 15:30:44 )...   22.05.2011, 22:30:14
- - konole   Okej, uploadowałem obrazek i chciałbym go wstawić ...   23.05.2011, 08:06:23
- - Daiquiri   Cytat(cisu @ 21.05.2011, 17:11:18 ) C...   23.05.2011, 08:27:18
- - 1010   Ale w przypadku, gdy upload odbywa się bez żadnego...   23.05.2011, 12:14:40
- - cisu   Witam, daję do sprawdzenia i oceny wersję beta moj...   10.09.2011, 11:13:48
- - Helid   Wykonanie ok (podoba mi się - wszystko proste w ob...   10.09.2011, 11:21:06
- - Rid   A jak tam ze sprawdzaniem heurastyki??Można każdy ...   10.09.2011, 11:33:14
|- - mortus   Cytat(Rid @ 10.09.2011, 12:33:14 ) A ...   10.09.2011, 11:49:52
|- - cisu   Cytat(mortus @ 10.09.2011, 12:49:52 )...   10.09.2011, 12:21:34
- - cisu   Ja nie chcę, żeby Wam się to przydało Piszę to pr...   10.09.2011, 11:33:42
- - toaspzoo   Widzę, że jest szyfrowanie hasła nawet ;P   10.09.2011, 12:00:56
- - Daiquiri   Połączyłam Twoje tematy. Regulamin: CytatWszelki...   10.09.2011, 13:41:26
|- - cisu   Cytat(Daiquiri @ 10.09.2011, 14:41:26...   10.09.2011, 18:12:47
- - !*!   - brak konta testowego - błędny link aktywacyjny p...   10.09.2011, 15:45:01
|- - cisu   Cytat(!*! @ 10.09.2011, 16:45...   10.09.2011, 18:06:31
- - Rid   Cytat@up występowanie 0 powoduje kolizje w pisaniu...   10.09.2011, 18:12:45
- - !*!   Robisz wersję prezentacyjną i udostępniasz zmianę ...   10.09.2011, 18:30:17
- - Daiquiri   Cytat(cisu @ 10.09.2011, 19:12:47 ) T...   11.09.2011, 07:07:53
- - marekge   To tylko fragment bardzo prostego kodu...niczym si...   13.09.2011, 11:33:22
- - lukaskolista   Nie moge zalogowac sie na konto demo - "Podan...   14.09.2011, 07:26:32
- - cisu   Komuś niestety przeszkadza domyślne hasło a ja nie...   14.09.2011, 14:51:04
- - Fifi209   Bo kto udostępnia zmianę hasła dla konta testowego...   22.09.2011, 14:54:08
- - cisu   Jak już prędzej napisałem, to jest normalne konto,...   22.09.2011, 16:07:46
- - !*!   cisu - rejestracja nie działa, już Ci to pisałem w...   22.09.2011, 16:41:46


Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 12.10.2025 - 05:41