Jump to content
Дизайн и модификация IPS Community IPBSkinsBETA
Search In
  • More options...
Find results that contain...
Find results in...
Sign in to follow this  
siv1987

Плагин авторизации через VK для IPS 4.x

Recommended Posts

Да, права есть.

 

Еще один момент, который хотелось бы исправить. На сайтах HTTPS импортированная аватарка грузится по HTTP, чем вызывает предупреждения в браузере.

 

З.Ы. Я вообще думал, аватарки скачиваются, а не вставляются по ссылке.

Share this post


Link to post
Share on other sites
Еще один момент, который хотелось бы исправить. На сайтах HTTPS импортированная аватарка грузится по HTTP, чем вызывает предупреждения в браузере.

https://vk.com/dev/api_requests

https – 1 — позволяет получать https ссылки на фотографии и другое медиа. 0 - методы возвращают http ссылки (по умолчанию). Обратите внимание, что этот параметр работает только если запросы к API осуществляются через https.

 

applications/core/sources/ProfileSync/VK.php

			$response = \IPS\Http\Url::external( "https://api.vk.com/method/getProfiles?uid={$this->member->vk_id}&access_token={$this->member->vk_token}&fields=photo_max_orig" )->request()->get()->decodeJson();

замените на

			$response = \IPS\Http\Url::external( "https://api.vk.com/method/getProfiles?uid={$this->member->vk_id}&https=" . \IPS\Request::i()->isSecure() . "&access_token={$this->member->vk_token}&fields=photo_max_orig" )->request()->get()->decodeJson();

  • Upvote 1

Share this post


Link to post
Share on other sites

Спасибо! Хотелось бы еще, чтобы это было внесено в дистрибутив плагина :)

Share this post


Link to post
Share on other sites

Обязательно будет внесено.

Share this post


Link to post
Share on other sites

Что-то правка от newbie не возымела эффекта. Вновь зарегившийся юзер имеет аватарку по HTTP.

Share this post


Link to post
Share on other sites

\IPS\Request::i()->isSecure() попробуйте заменить на intval( \IPS\Request::i()->isSecure() ) либо принудительно задать прописав 1.

 

Поддержка https добавлена в дистрибутив, для обновления скачать дистрибутив и перезалить файлы на фтп.

Проверьте чтобы ваш форум правильно определял протокол по которому он работает.

Share this post


Link to post
Share on other sites

@siv1987, посмотрите крайнюю версию форума 4.1.6, в которой исправлена xss в импортируемых статусах. В данный плагин тоже надо внести изменения. 1 строчка кода.

Share this post


Link to post
Share on other sites
В FB есть опция не только импорта, но и экспорта статусов, правда там надо пройти какую-то проверку приложения. А вконтакте такая возможность вообще не предусмотрена?
Экспорт возможен только для standalone-приложений.

Share this post


Link to post
Share on other sites

версию форума 4.1.6, в которой исправлена xss в импортируемых статусах

Исправлено.

Share this post


Link to post
Share on other sites

Еще один момент. Если у юзера неквадратная аватарка (например, вытянутая по вертикали), то она такой и попадает на форум. Насколько я помню, VK API позволяет получать именно квадратные аватарки.

Share this post


Link to post
Share on other sites

Еще один момент. Если у юзера неквадратная аватарка (например, вытянутая по вертикали), то она такой и попадает на форум. Насколько я помню, VK API позволяет получать именно квадратные аватарки.

https://vk.com/dev/fields

В плагине используется photo_max_orig (applications/core/sources/ProfileSync/VK.php)

Поменяйте значение на нужное Вам.

Share this post


Link to post
Share on other sites

Сделал, как там написано, зашел в Login Handlers и никакого VKontakte там не нашел, все осталось, как было.

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

 

Вот у меня, например, юзер:

 

post-56753-0-98821800-1452062859_thumb.png

Share this post


Link to post
Share on other sites
Да, я это понимаю, но думаю, для форумной аватарки нужны именно квадратные. Т.е. я предлагаю сделать это значением по умолчанию. Едва ли кому-то нужны в качестве аватарок оригинальные большие фотографии, к тому же зачастую искаженные.

На некоторых форумах видел здоровые аватары. Завтра придет другой юзер и захочет другой тип аватарки.

 

Т.е. я предлагаю сделать это значением по умолчанию.
Можно добавить настройку в АЦ, и пусть админ выбирает. Но тут решать @siv1987

Share this post


Link to post
Share on other sites

Временами все-таки проскакивают свежие юзеры с аватарками по HTTP вместо HTTPS.

Share this post


Link to post
Share on other sites
Временами все-таки проскакивают свежие юзеры с аватарками по HTTP вместо HTTPS.
Попробуйте поставить принудительно брать данные по https

Share this post


Link to post
Share on other sites
Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.

×
×
  • Create New...