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
2 Stron V   1 2 >  
Start new topic
Odpowiedzi (1 - 19)
1010
post
Post #2





Grupa: Zarejestrowani
Postów: 749
Pomógł: 37
Dołączył: 3.10.2006

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


100kb? Co tu w takim razie oceniać? Upload to tylko formularz + move_uploaded_file?
Go to the top of the page
+Quote Post
Daimos
post
Post #3





Grupa: Zarejestrowani
Postów: 1 319
Pomógł: 118
Dołączył: 26.11.2003
Skąd: Lublin

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


Przy małych plikach może to przeoczyłem, ale nie ma postępu uploadu, więc faktycznie nic tutaj do oceny nie ma
Wrzuciłem obrazek i nie ma podglądu nawet, trzeba pobrać, żeby zobaczyć
Nie ma żadnych podstron, więc jak przedmówca napisał, tu jest tylko "move_uploaded_file", no i licznik pobrań
Hash przy linku do samego pobierania chyba trochę przesadzony, chyba, że przekazujesz tam też kawałki jakiś powieści (IMG:style_emoticons/default/smile.gif)

No i wrzuć duży plik, ja dałem kilka GB, robi się upload (IMG:style_emoticons/default/wink.gif) sprawdzasz objętość po otrzymaniu pliku, później wyrzucasz błąd, przydało by się coś z tym zrobić

Ten post edytował Daimos 21.05.2011, 19:29:40
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%)
-----


Co do 100kb to jak napisałem, jest to wersja testowa. Co do podstron to niby jakie tam mam dać podstrony? Co do podglądów obrazków, paska postępu i sprawdzania rozmiaru pliku przed wysłaniem - nad tym aktualnie pracuję. Na razie ukierunkowałem się bardziej na pliki jako pliki, nie jako obrazki, dokumenty, wideo itd.

Nikt konta nie założył, a tam jest spis wszystkich swoich plików i kilka opcji więcej, ale dobra.

Czekam na więcej.
Go to the top of the page
+Quote Post
toaspzoo
post
Post #5





Grupa: Zarejestrowani
Postów: 778
Pomógł: 84
Dołączył: 29.07.2010
Skąd: Gliwice / Pławniowice Mistrz niezmordowanej klawiatury.

Ostrzeżenie: (20%)
X----


Założyłem konto

Fajne to.

Mam pytanie, skrypt działa na bazie danych (blob), czy pliki są na ftp?
Go to the top of the page
+Quote Post
r4xz
post
Post #6





Grupa: Zarejestrowani
Postów: 673
Pomógł: 106
Dołączył: 31.12.2008

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


Cytat(cisu @ 21.05.2011, 20:35:27 ) *
Nikt konta nie założył, a tam jest spis wszystkich swoich plików i kilka opcji więcej, ale dobra.

czyli liczysz na reklamę i użytkowników?
konto testowe...
Go to the top of the page
+Quote Post
cisu
post
Post #7





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

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


Cytat(toaspzoo @ 21.05.2011, 21:51:05 ) *
Założyłem konto

Fajne to.

Mam pytanie, skrypt działa na bazie danych (blob), czy pliki są na ftp?

Baza MySQL, pliki są w podkatalogu skryptu, ale to mam zamiar zmienić (żeby były wyżej tak, żeby nie było do nich dostępu bezpośrednio z przeglądarki mimo, że teraz blokuje je .htaccess).

Cytat(r4xz @ 21.05.2011, 21:56:35 ) *
czyli liczysz na reklamę i użytkowników?
konto testowe...

(IMG:style_emoticons/default/baaasmiley.gif) lol na jaką reklamę(IMG:style_emoticons/default/questionmark.gif) każdy kto pisze w tym dziale ma zaraz chcieć reklamę? po zalogowaniu jest więcej opcji i chciałem tylko, żeby ktoś poza mną sprawdził, czy działają.

Ten post edytował cisu 21.05.2011, 23:04:18
Go to the top of the page
+Quote Post
Fifi209
post
Post #8





Grupa: Zarejestrowani
Postów: 4 655
Pomógł: 556
Dołączył: 17.03.2009
Skąd: Katowice

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


A ciężko założyć konto testowe? Jak założysz, będę testował.
Go to the top of the page
+Quote Post
konole
post
Post #9





Grupa: Zarejestrowani
Postów: 275
Pomógł: 32
Dołączył: 21.03.2006
Skąd: Warszawa

Ostrzeżenie: (20%)
X----


Czekaj, to u ciebie uploaduje się plik w jakim celu? Aby podać komuś linka do ściągnięcia? Toż szybciej już jest wysłać przez Skype'a, ew. Gadu-Gadu... Zrób coś, co będzie użyteczne, a nie 'fajne'.
Go to the top of the page
+Quote Post
cisu
post
Post #10





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

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


Cytat(konole @ 22.05.2011, 15:30:44 ) *
Czekaj, to u ciebie uploaduje się plik w jakim celu? Aby podać komuś linka do ściągnięcia? Toż szybciej już jest wysłać przez Skype'a, ew. Gadu-Gadu... Zrób coś, co będzie użyteczne, a nie 'fajne'.

Fakt, przy plikach wielkości 100kb może i tak, ale ten limit to liczba pobrana z bazy danych i jest taka mała bo to wersja testowa. Wy nic nie ogarniacie w ogóle.
Go to the top of the page
+Quote Post
konole
post
Post #11





Grupa: Zarejestrowani
Postów: 275
Pomógł: 32
Dołączył: 21.03.2006
Skąd: Warszawa

Ostrzeżenie: (20%)
X----


Okej, uploadowałem obrazek i chciałbym go wstawić teraz, tutaj na forum. I co? I i i... mam podać link do ściągnięcia zamiast go po prostu wstawić tutaj? Bez sensu.
Go to the top of the page
+Quote Post
Daiquiri
post
Post #12





Grupa: Administratorzy
Postów: 1 552
Pomógł: 211
Dołączył: 7.07.2009
Skąd: NJ




Cytat(cisu @ 21.05.2011, 17:11:18 ) *
Chodzi mi o działanie skryptu, czy nie ma błędów itd.
Trzymajmy się tego, OK? Chyba, że autor tematu poprosi o ocenę pomysłu i jego powodzenia.
Go to the top of the page
+Quote Post
1010
post
Post #13





Grupa: Zarejestrowani
Postów: 749
Pomógł: 37
Dołączył: 3.10.2006

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


Ale w przypadku, gdy upload odbywa się bez żadnego paska postępu itp to w takim rozwiązaniu nie ma co oceniać.
Go to the top of the page
+Quote Post
cisu
post
Post #14





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

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


Witam, daję do sprawdzenia i oceny wersję beta mojego skryptu do uploadu i udostępniania plików.
Adres: http://upload.icisu.net/
Kilka uwag, zanim coś napiszecie:
1. Proszę nie pisać o tym, że strona na js'ie to zły pomysł, przemyślałem to i zdania nie zmienię.
2. Captcha przy rejestracji jest tymczasowa, zmienię (pewnie na recaptcha).
3. Chodzi mi tu o sprawdzenie i ocenę przede wszystkim wysyłania plików i zarządzania plikami. Jest konto testowe, więc nie trzeba się rejestrować (ale proszę nie zmieniać hasła na tym koncie).
4. Oprócz JS do wysłania plików wymagany jest flash.
5. To jest strona testowa. Wielkość jednego pliku to maks. 5 MB. Proszę usuwać pliki po skończeniu testowania.

Testowałem całość na Chrome, Firefoxie i IE, nie powinno być większych problemów na żadnych przeglądarkach.

Proszę o opinie i sugestie, co warto zmienić.
Go to the top of the page
+Quote Post
Helid
post
Post #15





Grupa: Zarejestrowani
Postów: 280
Pomógł: 20
Dołączył: 12.12.2007
Skąd: 127.0.0.1

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


Wykonanie ok (podoba mi się - wszystko proste w obsłudze), jednak nie rozumiem koncepcji tego. Do czego miałby mi się to przydać skoro mam multi innych serwisów tego typu?
Go to the top of the page
+Quote Post
Rid
post
Post #16





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
cisu
post
Post #17





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

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


Ja nie chcę, żeby Wam się to przydało (IMG:style_emoticons/default/smile.gif)
Piszę to przede wszystkim na własny użytek, może kiedyś udostępnię kod (najpierw muszę jeszcze nad tym popracować nieco).
Dałem do oceny, żeby wiedzieć co nie działa, co poprawić itd.

Co do wirusów, ja się uczyłem php sam, aż tak daleko jeszcze nie zaszedłem.
Ale jak możecie, to ich nie wrzucajcie (IMG:style_emoticons/default/smile.gif)

Ten post edytował cisu 10.09.2011, 11:34:59
Go to the top of the page
+Quote Post
mortus
post
Post #18





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
toaspzoo
post
Post #19





Grupa: Zarejestrowani
Postów: 778
Pomógł: 84
Dołączył: 29.07.2010
Skąd: Gliwice / Pławniowice Mistrz niezmordowanej klawiatury.

Ostrzeżenie: (20%)
X----


Widzę, że jest szyfrowanie hasła nawet ;P
Go to the top of the page
+Quote Post
cisu
post
Post #20





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

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

 



RSS Aktualny czas: 24.08.2025 - 23:58