В поддержку по плагину Inactive User Deleter недавно написали, что Wordfence считает мой плагин уязвимым к CSRF.
Я всегда пытаюсь реагировать на запросы юзеров, но сначала мне показалось, что опять какая то параноидальная тулза что то там отрапортовала, а пользователи бьют тревогу.
Во-первых, все формы находятся в админке, т.е. туда просто не зайти без административного доступа, чтобы что то там сделать. Т.е. сначала надо сломать админку, а потом уже что то можно будет сделать.
Во-вторых, это же надо нацелиться на такой узкий сегмент сайтов, которые используют мой плагин, чтобы заниматься какой то хакерской движухой, нацеленной именно через эту брешь.
Но теоретически, действительно, если перехвачены сессионные данные пользователя, а он при этом админ, то гипотетически, если правильно сформировать запрос, то, возможно, получится выполнить действия, которое мой плагин осуществляет при отправке данных через свои формы. А одним из действий как раз является удаление пользователей, при этом массовое удаление.
WP предлагает использование классического приёма защиты, называется NONCE. Что расшифровывается как ‘number used once’.
При каждом выводе формы специальной функцией добавляется некий уникальный ключ, который надо будет проверить, принимая данные.
|
1 2 |
// выводит поля формы с уникальным ключом wp_nonce_field('my_key_prefix'); |
Повторная отправка пакета данных, с тем же ключом или без него, не будет обработана, стоит только включить проверку ключа:
|
1 |
check_admin_referer('my_key_prefix') |
Если NONCE не прошел проверку — дальнейший вывод страницы прерывается с 403 кодом и сообщением об ошибке, предотвращая тем самым какие либо действия.
Добавил wp_nonce_field во все формы, чтобы юзер был доволен. :)