Mój skrypt uploadu, proszę o opinie Opcje

[cisu]

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.

[1010]

100kb? Co tu w takim razie oceniać? Upload to tylko formularz + move_uploaded_file?

[Daimos]

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

[cisu]

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.

[toaspzoo]

Założyłem konto

Fajne to.

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

[r4xz]

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...

[cisu]

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).

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

[Fifi209]

A ciężko założyć konto testowe? Jak założysz, będę testował.

[konole]

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'.

[cisu]

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.

[konole]

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.

[Daiquiri]

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.

[1010]

Ale w przypadku, gdy upload odbywa się bez żadnego paska postępu itp to w takim rozwiązaniu nie ma co oceniać.

[cisu]

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ć.

[Helid]

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?

[Rid]

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)

[cisu]

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

[mortus]

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.

[toaspzoo]

Widzę, że jest szyfrowanie hasła nawet ;P

[cisu]

- 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

- 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ć?

- 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

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ć.

Widzę, że jest szyfrowanie hasła nawet ;P

Hasło jest hashowane, a gdzie to widzisz?

[Daiquiri]

Połączyłam Twoje tematy.

Regulamin:

Wszelkie kolejne wersje danego projektu proszę umieszczać w tym samym wątku, zgodnie z zasadą : jeden projekt = jeden wątek.

[!*!]

- brak konta testowego
- błędny link aktywacyjny podczas rejestracji
- linki menu nie działają
- "przepisz od tyłu" wtf?
- @up występowanie 0 powoduje kolizje w pisaniu
- brzydki wygląd

Ponadto Administracja zastrzega sobie prawo do wysłania wiadomości w charakterze informacyjnym (np. zmiany w regulaminie, usunięcie plików Użytkownika itd.), oraz reklamowym (sporadyczne wiadomości zawierające informacje o produktach, usługach itd.).

Administracja zastrzega sobie prawo do zmiany tego regulaminu bez wcześniejszego powiadamiania o tym Użytkowników. Zmieniony regulamin obowiązuje od chwili publikacji.
O rozstrzygnięciu wszystkich nieporozumień wynikających z braku odpowiednich wpisów w regulaminie decyduje Administracja.

Ten post edytował !*! 10.09.2011, 15:48:53

[cisu]

- brak konta testowego
- błędny link aktywacyjny podczas rejestracji
- linki menu nie działają
- "przepisz od tyłu" wtf?
- @up występowanie 0 powoduje kolizje w pisaniu
- brzydki wygląd

Konto testowe jest, dane na stronie główne.
Aktywacja działa, nieco bazę danych zmieniałem to pewnie dlatego, a napisałem, żeby testować zarządzanie plikami - po to też zrobiłem konto testowe.
Wszystkie linki w menu działają.
Napisałem, że captcha jest tymczasowa...
Grafika jest robocza i będzie zmieniona, też napisałem.

- @up występowanie 0 powoduje kolizje w pisaniu

Nie wiem o co chodzi za bardzo.
Ktoś zmienił hasło do konta testowego i dlatego nie działa, za minutę będzie działać znowu.
Następnym razem, zanim napiszesz, czytaj bo tylko offtop robisz.

A co do regulaminu to też chyba jest, że będzie zmieniony, ale akurat nie wiem, co jest dziwnego w tych punktach, które podałeś.

Ten post edytował cisu 10.09.2011, 18:07:34

[Rid]

@up występowanie 0 powoduje kolizje w pisaniu

!*! chodziło chyba o to,że iczbę 0 można pomylić z literą O,tak mi się wydaje.

Ten post edytował Rid 10.09.2011, 18:13:17

[cisu]

Połączyłam Twoje tematy.

Regulamin:

To jest całkiem inny projekt, napisany od 0 inny skrypt, ale niech będzie.

[!*!]

Robisz wersję prezentacyjną i udostępniasz zmianę hasła dla takiego konta? Może i jest, ale nadal zalogować się nie mogę. Ktoś zmienił hasło. Linki nie działają np kontakt i polityka prywatności. Aktywacja też w dalszym ciągu nie działa. Skoro captcha ma zostać zmieniona, to po co dałeś ją teraz w ogóle, szczególnie że celem testu jest co innego. To niby jak mamy to testować? Udostępniaj projekty z głową.

Ten post edytował !*! 10.09.2011, 18:31:37

[Daiquiri]

To jest całkiem inny projekt, napisany od 0 inny skrypt, ale niech będzie.

Inny projekt? Przecież napisałeś: "Witam, daję do sprawdzenia i oceny wersję beta mojego skryptu do uploadu i udostępniania plików.". Wcześniej wrzuciłeś również skrypt uploadu. Jeżeli jest to zupełnie inny projekt, a nie ten sam tylko przepisany od zera (ten sam projekt, inny kod) - mogę wydzielić temat.

Niestety nic ponadto nie napiszę, zważywszy na zmianę hasła.

[marekge]

To tylko fragment bardzo prostego kodu...niczym się to nie wyróżnia, codziennie setki osób piszą takie skrypciki po przeczytaniu swojej pierwszej książki o php ;-)
W zasadzie moja wtyczka w komunikatorze do wysyłki plików na serwer jest bardzie rozbudowana.

[lukaskolista]

Nie moge zalogowac sie na konto demo - "Podane hasło jest nieprawidłowe. Spróbuj ponownie.".

[cisu]

Komuś niestety przeszkadza domyślne hasło a ja nie mam jak teraz zmienić.

[Fifi209]

Bo kto udostępnia zmianę hasła dla konta testowego...?

[cisu]

Jak już prędzej napisałem, to jest normalne konto, takie jak wszystkie inne, a założyłem je, bo jak trzeba się rejestrować to nikt tego nie robi. Po prostu jakiś są gdzieś tu wieśniaki i nie zaprzeczajcie, bo prosiłem, żeby nie zmieniać, a taki na złość robi nie tyle mi, ale całej reszcie.

[!*!]

cisu - rejestracja nie działa, już Ci to pisałem wcześniej. Jak udostępniasz słabej jakości skrypt, to się nie dziw że jest tak a nie inaczej.