Jump to content

Загрузка изображений на свой сервер при указании внешних ссылок.


 Share

Recommended Posts

Приветствую!

 

Если ли решение для такой задачи:

Если юзер загружает в сообщение изображение с внешнего сайта, нужно чтобы это изображение закачивалось на мой сервер, и отдавалось на форуме уже с моего сервера и никак не зависло от внешнего?

 

Link to comment
Share on other sites

  • 6 months later...

 

community.invisionpower.com/files/file/7188-dp34-download-images/

Спасибо!

Купил, развернул на тестовом форуме. Вроде все развернулось, тестовая картинка загрузилась успешно. Увидел, что имя файла не изменяется. Решил проверить на простейший баг - загрузил с другого сайта другое изображение, но с таким же именем.

Ну и да, изображение в папке было перезаписано новым. То есть злоумышленник может напакостить, подменяя изображения в чужих постах, загружая в свои посты изображения с тем же именем. Как такой эпик фэйл можно было допустить при разработке я не очень понимаю.

Link to comment
Share on other sites

Я вот ни разу не программист, но решил посмотреть, что можно сделать.

В файле нашел место, где генерируется имя. Нагуглил, что в PHP случайное число можно получить функцией rand(). В итоге заменил

$imageFull = $imageName . '.' . $imageExt;

на

$imageFull = $imageName . '_' . rand() . '.' . $imageExt;

теперь при записи к файлу добавляется случайное число и перезаписи не происходит.

Конечно, костыль, но работает. Больше беспокоит, что если там такие очевидные косяки, не нарвусь ли я с этим модом еще кна какие-нибудь грабли.

Link to comment
Share on other sites

В файле нашел место, где генерируется имя. Нагуглил, что в PHP случайное число можно получить функцией rand(). В итоге заменил

Вместо rand() используйте лучше что-нибудь типа 

md5( uniqid( microtime(), true ) )

 

Edited by LvsF
Link to comment
Share on other sites

@polym0rph вы автору написали об этом баге?  

Да, на странице приложения invisionpower.com, чтобы заодно и пользователей предупредить.  Но сообщение еще пока не прошло местную модерацию.

Я так подумал, можно на форуме, которую такую штуку использует, пройтись парсером, собрать все имена картинок, и скриптом их все заменить, подгружая какую-то одну, изменяя ее имя. Неприятно очень будет для форума.

Вместо rand() используйте лучше что-нибудь типа 

md5( uniqid( microtime(), true )

Спасибо!

Link to comment
Share on other sites

Да, на странице приложения invisionpower.com, чтобы заодно и пользователей предупредить.  Но сообщение еще пока не прошло местную модерацию.

Можете в теме поддержки файла написать. Там премодерации нет.

Я так подумал, можно на форуме, которую такую штуку использует, пройтись парсером, собрать все имена картинок, и скриптом их все заменить, подгружая какую-то одну, изменяя ее имя. Неприятно очень будет для форума.

Не понял. Судя по скриншоту, в приложении есть инструмент для этого.

Link to comment
Share on other sites

Можете в теме поддержки файла написать. Там премодерации нет.

Не понял. Судя по скриншоту, в приложении есть инструмент для этого.

1. Да, там тоже написал.

2. Это скорее сценарий для атаки, как форуму навредить, где такой мод есть. Кстати, внутренним инструментом пробовал заменять, постоянно на тестовом сервере вываливается, даже при 5 элементах в очереди, вместо дефолтных 50.

Link to comment
Share on other sites

md5( uniqid( microtime(), true )

не  хочет, в логах

PHP Parse error:  syntax error, unexpected ';'

Покурил мануалы, остановился на

md5(uniqid(rand(),1))

так все работает.

Link to comment
Share on other sites

не  хочет, в логах

PHP Parse error:  syntax error, unexpected ';'

Покурил мануалы, остановился на

md5(uniqid(rand(),1))

так все работает.

Закрывающая скобка похерена была.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Ваша ссылка была автоматически строена.   Отображать как обычную ссылку

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...