Changing login url

I think the article’s logout hook is wrong on two or three counts:

• it needs ?action=logout, similar to the lost password link
• it needs a generated nonce too
• it doesn’t respect the $redirect argument.

Here’s a new version based on the current wp_logout_url() code:

add_filter( 'logout_url', 'my_logout_page', 10, 2 );
function my_logout_page( $logout_url, $redirect ) {
    $args = array();
    if ( ! empty( $redirect ) ) {
        $args['redirect_to'] = urlencode( $redirect );
    }

    $logout_url = add_query_arg( $args, site_url( 'my-secret-login.php?action=logout', 'login' ) );
    $logout_url = wp_nonce_url( $logout_url, 'log-out' );

    return $logout_url;
}