Забыли пароль?
Запросите новый здесь.

Автор темы: Polarfox
ID темы: 2282
Эта тема прикреплена!
В данной теме может содержаться важная информация. Имеются прикрепленные файлы.
Просмотр темы
PHP-Fusion Russia » Поддержка 7 версии » Ошибки, баги
 Распечатать тему
ВАЖНО - Изменение в IMG версии 7.02.07
Polarfox
Про bbcode IMG версии 7.02.07
Важно прочитать внимательно, а дальше - сами решаете что делать.
Все это имхо, я кулаком по столу в пьяном угаре не бью и не требую чтоб так оно и было, но просто поясню.


Как было в пред. версиях
Простой обработчик заменял по рег.выражению тег IMG на картинку ( <img src="???"> )

Что добавлено сейчас
Добавлен "патч" для... вот не знаю для чего, в теории - для безопасности.
Суть обновления - теперь сервер обязан уметь получать заголовки с удаленного сервера (с картинкой) через функцию getimagesize, по документации не написано - загружает ли она файл или берет заголовки (во втором варианте смысла в ней нет вообще, в первом - она успешно может повесить сервер).
Все это проходит в три шага:
  1. получаем данные о картинки (удаленной, она всегда начинается с http(s)/ftp(s) )
  2. проверяем размер (как проверка на картинку)
  3. если размер плох - это не картинка - так и пишет, если ок - выводим картинку


Минусы
  • замедление выдачи страницы при проблеме с картинкой - например если удаленный сервер не отвечает - ваш сервер будет ждать до таймаута а потом возвратит "ошибку картинки"
  • большая нагрузка на сервер (с множеством картинок)
  • сервер может сказать что картинка "не верна" если: он может загрузить ее - в случае медленного ответа сервера или запрета вообще загружать со стороны ( см. Google Search allow_url_fopen )
  • медленное отображение картинок
  • и все подобное, по вкусу добавить

Плюсы
  • Нет


Мое решение
Восстановить оригинальный файл версии 7.02.06


Сам файл находится по этому пути
\includes\bbcodes\img_bbcode_include.php

Файл версии 07
Обновление кода


Файл от 7.02.06 (адекватный) в прикреплении
Polarfox присоединено следующее:файл:
img_from_70206.zip [1.63кБ / 560 Загрузки]

Всегда делайте backup перед изменениями | Указывайте свою версию в подписи/профиле. | Вся бесплатная тех. поддержка только на форуме
x2
 
Rush
печаль конечно, getimagesize() насколько я знаю не грузит всю картинку, но работает один хрен довольно медленно. зачем такие костыли нужны вообще? секьюрности помоему не добавляет
x1
 
Web
SchreiBear
Rush, я даже видел на одном форуме ...там картинки подгружаются на сервер что бы ...если картинки не окажется на указанному адресу всё равно открыть её...ну уже из своего запасника...только зачем хранить эту макулатуру не пойму ???
Нужно сказать огромное спасибо людям, которые живут на популярных девелопер форумах и дают там адекватные ответы на абсолютно «нубские» вопросы без попыток унизить вопрошающего.
 
Polarfox
Но у нас не так, в этом и суть.
Всегда делайте backup перед изменениями | Указывайте свою версию в подписи/профиле. | Вся бесплатная тех. поддержка только на форуме
 
Rush
для этого есть cloudflare и подобные
 
Web
SchreiBear
PolarFox, $avdata = @getimagesize("$matches[1]$matches[3]$matches[4]"ad;

по идеи тут заглушка стоит @ так что не каких ошибок быть не должно
Нужно сказать огромное спасибо людям, которые живут на популярных девелопер форумах и дают там адекватные ответы на абсолютно «нубские» вопросы без попыток унизить вопрошающего.
 
Polarfox
Если ты внимательно прочтешь код, то увидишь что по возвращаемому значению [IF] может появиться текст [... is not a valid Image], где как раз и есть сообщение об ошибке.

Здесь вообще речь не о собаке как таковой, перечитай минусы.
Всегда делайте backup перед изменениями | Указывайте свою версию в подписи/профиле. | Вся бесплатная тех. поддержка только на форуме
 
Alex
раньше при наличии в теме много картинок вставленных с других серверов, приходилось долго ждать открытия страницы, это наверно было не хорошо.

загрузил файлы из первого поста, страницы стали открываться быстро!
 
Web
Polarfox
Необоснованное "улучшение" ,к сожалению , портит жизнь пользователям, мне часто очень не хочется быть правым, но...
Всегда делайте backup перед изменениями | Указывайте свою версию в подписи/профиле. | Вся бесплатная тех. поддержка только на форуме
 

Поделиться этой темой
Социальные закладки: Vkontakte Odnoklassniki Mail.ru Facebook Google Tweet This
URL:
BBcode:
HTML:

Перейти на форум:
Похожие темы
Топ 5 пользователей форума
Zaxap Zaxap (1,090)   Vova Vova (877)   Pisatel Pisatel (678)   util util (666)   SchreiBear SchreiBear (625)