Search
Write a publication
Pull to refresh

Comments 56

Уведомляю вас, так как вы подписаны на новости проекта:

Устранены жуткие баги:
1. Запрещена интерпретация PHP.
2. Теперь файлы .php,.cgi,.pl загружать нельзя.
может стоило обойтись запретом интерпретации и исполнения? файлообмен получается порепаным — не все можно закачать, придется извращаться с переименованием либо архиваторами
Ну, не так уж много и запрещено — всего-то три типа. Если у вас будет специфическая необходимость загружать именно эти файлы — можете просто убрать проверку. Могу вынести список запрещенных расширений в конфиг для удобства. Что скажете?
Да и сам думаю, что так будет во всех смыслах лучше. Исправлю.
Исправил — теперь список запрещенных расширений в конфиге.
Не понимаю, почему этот комментарий минусуют.
Может быть потому, что
Invalid user. Please login
неправда, спс за «Short_-_Modern_Web_Standards.pdf»
Уже всё ок. Тогда не пускало почему-то.
Спасибо за обнаружение бага. Если не трудно, подскажите решение.
Спасибо большое! Помогло. Все исправил, архив перезалил.
что делает этот код:
if($username >= '!' && $password >= '!')
{
  setcookie('username',$username);
  setcookie('password',$password);
}

почему такое использовано такое странное условия для проверки наличия символов в строке?
почему не пароль не шифруется?
Предложите пожалуйста другие условия. С удовольствием их поменяю на ваши, если они будут более правильны. Пароли не шифруются, потому что для меня это не было актуально.
Ну, я все же думаю, что username, да и password тоже, должен быть больше, чем пробел (код 32). Ну и, само собой, больше чем восклицательный знак (код 33). Но это мое мнение. А по большому счету, сойдет конечно и empty…
Вы ведь не регистрацию пользователю проводите, а лишь принимаете данные, которые затем сравниваете с данными в БД (логин/пароль). Зачем тут такие сложности, не пойму. В популярный cms/системах, написанных на фреймворках довольно часто встречается приблизительно следующий код:

$model = User::findByLogin($username);
if($model && $model->checkPassword($password))
{
    User::login($model);
}
Да, это проверка. Но о какой сложности вы говорите? Что может быть проще сравнения строк?
Это очень важно и для нас всех тут присутствующих! Весьма захватывающая статья!
Я к нему привык. Он меня полностью устраивает.
Спасибо за минус к комменту — я к ним тоже привык. Они меня тоже полностью устраивают.
Исходник под какой лицензией? Юнит-тесты есть?
Лицензию сам себе какую-нибудь придумайте — мне пофиг.
Для начала не стоит давать загружать php. И не стоит выполнять скрипты (в тч php) в загруженных файлах. Настройте наконец ваш Апач.

Теперь рассмотрим ваш phpinfo. Для сохранности он помещен в архив (пока вам умельцы не залили шел и все не снесли, например).
Использовать в php.ini auto_append_file и auto_prepend_file — плохая идея.
А вот стоит ли в таком проекте держать включенным allow_url_fopen — сомнительно.

Держать в куках логин и пароль прямым текстом — плохая идея.
Спасибо, за развернутый список недочетов. Я им обязательно займусь. Интерпретацию php я уже исправил — еще раз спасибо DYPA.
вот такие люди, как автор, улучшают жизнь остальным! спасибо, именно то что я искал годы.
Рад, что вам пригодилось. Собственно, я разместил исходники именно здесь только ради этого. Я конечно, понимаю, что багов в сем творении много — писалось давно. Но оно работает — это раз. А исправить баги для профи — не проблема, учитывая небольшой объем кода — это два. Как-то так…
Всем отдельное спасибо за дельные комментарии :-)
Кажется, зажигается новая звезда Хабра.
Бумбурум, Мицгол, Ализар… У каждого свой путь к популярности, свой стиль.
smkuzmin выбрал путь «великого велосипедиста».

Все это как минимум позитивно :)
А мне нравиться холиварная фраза «Веб-макаки — не программисты»=)
может ПоХаПэ — это диагноз?
Напишите тоже самое, только на c++. Я вас очень прошу…
Зачем?
Вы что-то хотите этим сказать? Прошу вас больше конкретики.
А что вы хотели сказать? Что — те, кто пишет на PHP — макаки, а те, кто на C++ — программисты? Поконкретнее, прошу вас.
ПоХаПэ != Конкретный_ПоХаПист, не?)
Простите пожалуйста, я погорячился.
Проблема в том, что если бы этот пост не был размещен в «Программирование» я бы его даже не заметил.
Но простите, что общего между например лямбда-кубом и этой хренью описанным в статье?

Я хочу сказать, что теза «Веб-макаки — не программисты» как никогда хорошо выражается этим случаем.

ПоХаПе, Веб-разработка — пожалуйста, но не Программирование.
Извините, что задел вашу профессиональную гордость программиста своей хренью.
Sign up to leave a comment.

Articles