Alternative hosts for thumbs

GhostUser
Posts: 47
Joined: Thu Mar 25, 2021 10:30 pm

Alternative hosts for thumbs

Post by GhostUser »

собственно, сейчас, если указать Alternative hosts for thumbs и Alternative hosts for movie (flv) urls то скрипт после указанного занчения добавляет захардкоденное scj/thumbs. это не даёт возможности настроить сервак на scj/thumbs как на корень, т.е. что б было просто так cdn.example.com/galleries/0/010/0_t.jpg (без scj/thumbs) не получится.

поэтому, предлагаю убрать из скрипта захардкоденное scj/thumbs, а спецификацию полного пути делегировать юзеру, т.е. что бы он явно сам указывал полный путь, типа cdn.example.com/scj/thumbs/- урлы на тумбы генерятся как раньше, cdn.example.com - без этого постфикса и т.д.

так же, для поддержки совместимости с текущими инсталлами, перед этими полями (Alternative hosts for thumbs) добавляется одна галка (которая по дефолту выключена), типа:
[x] - i'll specify full path to the thumbnail directory by myself

и если она включена, скрипт не добавляет scj/thumbs к тому что указано в этих полях, а для формирования урла юзает только то, что указано в этих полях. если выключена (по дефолту), то делает как раньше.
admin
Site Admin
Posts: 37202
Joined: Wed Sep 10, 2008 11:43 am

Re: Alternative hosts for thumbs

Post by admin »

scj/thumbs

нигде не законедо, то в grabber settings прописано
Don't forget to run script update
GhostUser
Posts: 47
Joined: Thu Mar 25, 2021 10:30 pm

Re: Alternative hosts for thumbs

Post by GhostUser »

а, нашёл теперь...
ок, можно закрывать
admin
Site Admin
Posts: 37202
Joined: Wed Sep 10, 2008 11:43 am

Re: Alternative hosts for thumbs

Post by admin »

ok
Don't forget to run script update
GhostUser
Posts: 47
Joined: Thu Mar 25, 2021 10:30 pm

Re: Alternative hosts for thumbs

Post by GhostUser »

кстати, возник ещё такой момент - если мы вешаем на https:// то схема почему-то сама подставляется как http://
т.е. заходим на сайт, как на https://example.com, но урлы на тумбе генерятся как http://cdn.example.com
имхо нужно что б скрипт смотрел на $_SERVER['REQUEST_SCHEME'] (или $_SERVER['HTTPS']) и автоматом юзал это значение, или требовать добавлять схему явно в Alternative hosts for thumbs (и юзать как сейчас, если она не задана для обратной совместимости)

те. если в Alternative hosts for thumbs указано просто cdn.example.com, то работает как сейчас (для обратной совместимости), если https://cdn.example.com, то для формирования урлы на ресурсы используется указанная схема

а если ни Alternative hosts for thumbs ни Alternative hosts for movie (flv) urls не указаны (или заданы без схемы), то смотрим $_SERVER['REQUEST_SCHEME']
admin
Site Admin
Posts: 37202
Joined: Wed Sep 10, 2008 11:43 am

Re: Alternative hosts for thumbs

Post by admin »

//cdn.example.com

надо прописать
Don't forget to run script update
GhostUser
Posts: 47
Joined: Thu Mar 25, 2021 10:30 pm

Re: Alternative hosts for thumbs

Post by GhostUser »

угу, ну я тестил schemeless-урлы, мало-ли... хорошо что уточнили что скрипт если видит такое, то сам схему не подставляет
ок тогда

зы: на самом форуме стал часто вылазить ворнинг в самом верху:

Code: Select all

[phpBB Debug] PHP Warning: in file [ROOT]/ext/blitze/sitemaker/blocks/my_block.php on line 91: count(): Parameter must be an array or an object that implements Countable
ну то такое - что б новый топик не открывать )
admin
Site Admin
Posts: 37202
Joined: Wed Sep 10, 2008 11:43 am

Re: Alternative hosts for thumbs

Post by admin »

Спасибо, добавил проверочку)
Don't forget to run script update
GhostUser
Posts: 47
Joined: Thu Mar 25, 2021 10:30 pm

Re: Alternative hosts for thumbs

Post by GhostUser »

ещё <!--MODEL_PHOTO--> выбираются без учёта Alternative hosts. те. /model_images/0/010.jpg вместо //cdn.example.com/model_images/0/010.jpg
admin
Site Admin
Posts: 37202
Joined: Wed Sep 10, 2008 11:43 am

Re: Alternative hosts for thumbs

Post by admin »

сорри этот таг уже убрали , я забыл удалить из вики

<models num=1-10>

<!--MODEL_NAME-->
<model_photos num=1-10> <!--THUMB_URL--> </model_photos>
<br>

</models>
Don't forget to run script update
Post Reply