Внедрение JS в phpBB

Идеи для расширения функциональности phpBB 2.0.x
Аватара пользователя
Vovochka
phpBB 1.2.0
Сообщения: 16
Стаж: 19 лет 11 месяцев

Сообщение Vovochka »

Ага, нашел. Действительно, сверяет IP, но не полностью, а только 6 знаков из 8 (в 16-ричном виде). Теперь понятно, почему дома все так замечательно работает...

Здорово, что я был не прав.
Но ненормальную аватарку я-то на форум загрузил. И, хоть кто-то, да по моей ссылке перешёл...
Аватара пользователя
Mr. Anderson
phpBB Guru
phpBB Guru
Сообщения: 7522
Стаж: 21 год 1 месяц
Откуда: СССР
Благодарил (а): 4 раза
Поблагодарили: 94 раза

Сообщение Mr. Anderson »

Vovochka писал(а):Но ненормальную аватарку я-то на форум загрузил. И, хоть кто-то, да по моей ссылке перешёл...
Полагаю, что достаточно просто отключить удаленных аватар. А по ссылке прошли для эксперимента ;)
Аватара пользователя
Balamut
Former team member
Сообщения: 2214
Стаж: 20 лет 1 месяц
Откуда: {postrow.POSTER_FROM}
Поблагодарили: 68 раз

Сообщение Balamut »

Я прошёл по ссылке. Действительно
Mr. Anderson писал(а):для эксперимента
После чего сменил пасс. :-)))) На всякий. ;-)))
//
// That's all, Folks!
// -------------------------------------------------
Аватара пользователя
Vovochka
phpBB 1.2.0
Сообщения: 16
Стаж: 19 лет 11 месяцев

Сообщение Vovochka »

Mr. Anderson писал(а): Полагаю, что достаточно просто отключить удаленных аватар
Нет.
Надо запретить закачку аватар. Ведь в том и дело, что файл находится именно в домене форума, следовательно, есть доступ к cookie...

Я запретил закачку аватар. Но форум перестал показывать уже закачанные.
Поэтому в viewtopic.php

Код: Выделить всё

		switch( $postrow[$i]['user_avatar_type'] )
		{
			case USER_AVATAR_UPLOAD:
				$poster_avatar = ( $board_config['allow_avatar_upload'] ) ? '<img src="' . $board_config['avatar_path'] . '/' . $postrow[$i]['user_avatar'] . '" alt="" border="0" />' : '';
				break;
			case USER_AVATAR_REMOTE:
				$poster_avatar = ( $board_config['allow_avatar_remote'] ) ? '<img src="' . $postrow[$i]['user_avatar'] . '" alt="" border="0" />' : '';
				break;
			case USER_AVATAR_GALLERY:
				$poster_avatar = ( $board_config['allow_avatar_local'] ) ? '<img src="' . $board_config['avatar_gallery_path'] . '/' . $postrow[$i]['user_avatar'] . '" alt="" border="0" />' : '';
				break;
		}
надо заменить на

Код: Выделить всё

		switch( $postrow[$i]['user_avatar_type'] )
		{
			case USER_AVATAR_UPLOAD:
				$poster_avatar = '<img src="' . $board_config['avatar_path'] . '/' . $postrow[$i]['user_avatar'] . '" alt="" border="0" />';
				break;
			case USER_AVATAR_REMOTE:
				$poster_avatar = '<img src="' . $postrow[$i]['user_avatar'] . '" alt="" border="0" />';
				break;
			case USER_AVATAR_GALLERY:
				$poster_avatar = '<img src="' . $board_config['avatar_gallery_path'] . '/' . $postrow[$i]['user_avatar'] . '" alt="" border="0" />';
				break;
		}
А в usercp_viewprofile.php

Код: Выделить всё

	switch( $profiledata['user_avatar_type'] )
	{
		case USER_AVATAR_UPLOAD:
			$avatar_img = ( $board_config['allow_avatar_upload'] ) ? '<img src="' . $board_config['avatar_path'] . '/' . $profiledata['user_avatar'] . '" alt="" border="0" />' : '';
			break;
		case USER_AVATAR_REMOTE:
			$avatar_img = ( $board_config['allow_avatar_remote'] ) ? '<img src="' . $profiledata['user_avatar'] . '" alt="" border="0" />' : '';
			break;
		case USER_AVATAR_GALLERY:
			$avatar_img = ( $board_config['allow_avatar_local'] ) ? '<img src="' . $board_config['avatar_gallery_path'] . '/' . $profiledata['user_avatar'] . '" alt="" border="0" />' : '';
			break;
	} 
заменить на

Код: Выделить всё

	switch( $profiledata['user_avatar_type'] )
	{
		case USER_AVATAR_UPLOAD:
			$avatar_img = '<img src="' . $board_config['avatar_path'] . '/' . $profiledata['user_avatar'] . '" alt="" border="0" />';
			break;
		case USER_AVATAR_REMOTE:
			$avatar_img = '<img src="' . $profiledata['user_avatar'] . '" alt="" border="0" />';
			break;
		case USER_AVATAR_GALLERY:
			$avatar_img = '<img src="' . $board_config['avatar_gallery_path'] . '/' . $profiledata['user_avatar'] . '" alt="" border="0" />';
			break;
	}
После этого будут отображаться все аватарки, которые были получены форумом до установки запрета на их получение.

Еще я добавил в админпанель пункт о "глобальном" разрешении/запрете показа аватар.
Аватара пользователя
Mr. Anderson
phpBB Guru
phpBB Guru
Сообщения: 7522
Стаж: 21 год 1 месяц
Откуда: СССР
Благодарил (а): 4 раза
Поблагодарили: 94 раза

Сообщение Mr. Anderson »

Vovochka писал(а):Надо запретить закачку аватар
А зачем, если проще не юзать ИЕ? :)
Хотя разумнее прописать более "глубокую" проверку закачиваемого... только и всего.
Аватара пользователя
Vovochka
phpBB 1.2.0
Сообщения: 16
Стаж: 19 лет 11 месяцев

Сообщение Vovochka »

Mr. Anderson писал(а):А зачем, если проще не юзать ИЕ? :)
Хотя разумнее прописать более "глубокую" проверку закачиваемого... только и всего.
Вот, именно эту "глубокую" проверку я и "запрашиваю".

Я просто предложил временную затычку. Но для изменения аватар либо что-то дописывать, либо базу ручками править и файлики качать.
Аватара пользователя
Romiyo
Неадекватен
Сообщения: 603
Стаж: 20 лет 4 месяца
Откуда: tln.ee

Сообщение Romiyo »

Vovochka то есть вы хотите сказать, что с помощью закачки аватарок с компа на сервак можно такую же мулю сотворить с кукисами?
Неадекватен
Аватара пользователя
Vovochka
phpBB 1.2.0
Сообщения: 16
Стаж: 19 лет 11 месяцев

Сообщение Vovochka »

Romiyo,
Я просто могу получить cookies.
Как мне уже разъяснили, многое я не сделаю...

Но я не понял вопрос.

Куки phpBB хороши тем, что в них, как я понял (если правильно), хэш пароля и сам пароль не хранится. Правда, с флажком "запомнить меня", я не баловался.

Добавлено спустя 3 минуты 11 секунд:

Да и вообще, любой JS код выполнить могу (если человек по ссылке пойдет)...
Аватара пользователя
Romiyo
Неадекватен
Сообщения: 603
Стаж: 20 лет 4 месяца
Откуда: tln.ee

Сообщение Romiyo »

Romiyo писал(а):то есть вы хотите сказать, что с помощью закачки аватарок с компа на сервак можно такую же мулю сотворить с кукисами?
Неадекватен
Аватара пользователя
Vovochka
phpBB 1.2.0
Сообщения: 16
Стаж: 19 лет 11 месяцев

Сообщение Vovochka »

Romiyo,
Vovochka писал(а):Я смог закачать gif файл с содержанием:
<script>alert(document.cookie);</script>
А затем выполнил.
Vovochka писал(а):Резутьтат [удалено]
Всем:
Кто знает, как
Mr. Anderson писал(а):прописать более "глубокую" проверку закачиваемого... только и всего.
Аватара пользователя
Гога
phpBB 2.0.0
Сообщения: 220
Стаж: 19 лет 11 месяцев
Откуда: Архангельск

Сообщение Гога »

Vovochka писал(а):Я смог закачать gif файл с содержанием:
<script>alert(document.cookie);</script>
Это возможно только с GIF?
Гугл рулит. Я люблю ЛОР.
Интересно, они пришли с миром или им таки нужен наш моск?
Знания принадлежат всему человечеству.
Аватара пользователя
Vladson
Former team member
Сообщения: 816
Стаж: 21 год
Откуда: Estonia, Tallinn

Сообщение Vladson »

Серый цвет - светлый (светлее чёрного), но он и тёмный (темнее белого), он же промежуточный (между чёрным и белым). Теорию относительности никто не отменял. Истина в целом - понятие виртуально-ситуативное.
Аватара пользователя
Vovochka
phpBB 1.2.0
Сообщения: 16
Стаж: 19 лет 11 месяцев

Сообщение Vovochka »

Спасибо.
Я уже все исправил.

Вернуться в «Запросы модов для phpBB 2.0.x»