Удаление имени пользователя из файла cookie «wordpress_logged_in»

9

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

wordpress_logged_in[username]|[hash]

это то, что должно быть удалено. Поскольку это неотъемлемая часть системы входа в систему, я не уверен, как ее удалить и все еще поддерживать сеанс.

phatskat
источник

Ответы:

10

Краткое введение

После быстрого просмотра исходного кода WP, я думаю, что нашел решение ...

WordPress использует две функции для установки и анализа файлов cookie:

  • wp_generate_auth_cookie
  • wp_parse_auth_cookie

В wp_generate_auth_cookieвызванном фильтре есть фильтр, auth_cookieкоторый вы, вероятно, можете использовать для изменения содержимого cookie, но внутри фильтра нет wp_parse_auth_cookie, но ...

Обе эти функции определены в pluggable.php, что означает, что вы можете написать свои собственные реализации для них и переписать стандартные.

Решение

  1. Напишите свой собственный плагин (назовем его Better Auth Cookie)
  2. Реализуйте свои собственные wp_generate_auth_cookieи wp_parse_auth_cookieфункции внутри этого плагина.
  3. Активируйте свой плагин.

Вы можете найти мой пример реализации (строго основанный на оригинальных версиях) этих функций ниже:

if ( !function_exists('wp_generate_auth_cookie') ) :
/**
 * Generate authentication cookie contents.
 *
 * @since 2.5.0
 *
 * @param int $user_id User ID
 * @param int $expiration Cookie expiration in seconds
 * @param string $scheme Optional. The cookie scheme to use: auth, secure_auth, or logged_in
 * @param string $token User's session token to use for this cookie
 * @return string Authentication cookie contents. Empty string if user does not exist.
 */
function wp_generate_auth_cookie( $user_id, $expiration, $scheme = 'auth', $token = '' ) {
    $user = get_userdata($user_id);
    if ( ! $user ) {
        return '';
    }

    if ( ! $token ) {
        $manager = WP_Session_Tokens::get_instance( $user_id );
        $token = $manager->create( $expiration );
    }

    $pass_frag = substr($user->user_pass, 8, 4);

    $key = wp_hash( $user->user_login . '|' . $pass_frag . '|' . $expiration . '|' . $token, $scheme );

    // If ext/hash is not present, compat.php's hash_hmac() does not support sha256.
    $algo = function_exists( 'hash' ) ? 'sha256' : 'sha1';
    $hash = hash_hmac( $algo, $user->user_login . '|' . $expiration . '|' . $token, $key );

    $cookie = $user_id . '|' . $expiration . '|' . $token . '|' . $hash;

    /**
     * Filter the authentication cookie.
     *
     * @since 2.5.0
     *
     * @param string $cookie     Authentication cookie.
     * @param int    $user_id    User ID.
     * @param int    $expiration Authentication cookie expiration in seconds.
     * @param string $scheme     Cookie scheme used. Accepts 'auth', 'secure_auth', or 'logged_in'.
     * @param string $token      User's session token used.
     */
    return apply_filters( 'auth_cookie', $cookie, $user_id, $expiration, $scheme, $token );
}
endif;


if ( !function_exists('wp_parse_auth_cookie') ) :
/**
 * Parse a cookie into its components
 *
 * @since 2.7.0
 *
 * @param string $cookie
 * @param string $scheme Optional. The cookie scheme to use: auth, secure_auth, or logged_in
 * @return array Authentication cookie components
 */
function wp_parse_auth_cookie($cookie = '', $scheme = '') {
    if ( empty($cookie) ) {
        switch ($scheme){
            case 'auth':
                $cookie_name = AUTH_COOKIE;
                break;
            case 'secure_auth':
                $cookie_name = SECURE_AUTH_COOKIE;
                break;
            case "logged_in":
                $cookie_name = LOGGED_IN_COOKIE;
                break;
            default:
                if ( is_ssl() ) {
                    $cookie_name = SECURE_AUTH_COOKIE;
                    $scheme = 'secure_auth';
                } else {
                    $cookie_name = AUTH_COOKIE;
                    $scheme = 'auth';
                }
        }

        if ( empty($_COOKIE[$cookie_name]) )
            return false;
        $cookie = $_COOKIE[$cookie_name];
    }

    $cookie_elements = explode('|', $cookie);
    if ( count( $cookie_elements ) !== 4 ) {
        return false;
    }

    list( $user_id, $expiration, $token, $hmac ) = $cookie_elements;

    $user = get_userdata($user_id);
    $username = ( ! $user ) ? '' : $user->user_login;

    return compact( 'username', 'expiration', 'token', 'hmac', 'scheme' );
}
endif;

Моя версия этих функций заменяется user_loginна user_id. Но это должно быть хорошим началом для изменения его на что-то еще более сложное (то есть пользовательский хэш или что-то вроде этого).

Krzysiek Dróżdż
источник
Хороший ответ. Хотя я буду ждать до последнего дня моего щедрости. :)
Анонимный утконос
Я собираюсь принять это, хотя я не буду проверять это, поскольку мне больше не нужно это решение. Вы явно приложили немало усилий, чтобы покопаться в корне системы, я ценю это усилие :)
phatskat
1
Хотя этот подход хорош, вы должны знать, что он больше не защищает. Имя пользователя заменяется идентификатором пользователя, но имя пользователя может быть получено из идентификатора пользователя через запрос к example.com?author=123, который выполняет каноническое перенаправление на URL-адрес, такой как example.com/author/john.
Джон Блэкборн
1
@ Джон, пожалуйста, внимательно прочитайте. Я упомянул, что вы можете легко сделать намного более безопасным хранение некоторого случайного хэша в cookie вместо userID.
Krzysiek Dróżdż