WP登录尝试

Question

I've always had policy ' the less plugins the better ', so please don't suggest Login Attempts plugin, especially if it haven't been updated for several years.

That being said I found this snippet & added it to functions.php :

//Limit Login Attempts
if ( ! class_exists( 'limit_login_attempts' ) ) {
    class limit_login_attempts {

        var $failed_login_limit = 3;                    //Number of authentification accepted
        var $lockout_duration   = 1800;                 //Stop authentification process for 30 minutes: 60*30 = 1800
        var $transient_name     = 'attempted_login';    //Transient used

        public function __construct() {
            add_filter( 'authenticate', array( $this, 'check_attempted_login' ), 30, 3 );
            add_action( 'wp_login_failed', array( $this, 'login_failed' ), 10, 1 );
        }

        /**
         * Lock login attempts of failed login limit is reached
         */
        public function check_attempted_login( $user, $username, $password ) {
            if ( get_transient( $this->transient_name ) ) {
                $datas = get_transient( $this->transient_name );

                if ( $datas['tried'] >= $this->failed_login_limit ) {
                    $until = get_option( '_transient_timeout_' . $this->transient_name );
                    $time = $this->when( $until );

                    //Display error message to the user when limit is reached 
                    return new WP_Error( 'too_many_tried', sprintf( __( '<strong>ERROR</strong>: You have reached authentification limit, you will be able to try again in %1$s.' ) , $time ) );
                }
            }

            return $user;
        }


        /**
         * Add transient
         */
        public function login_failed( $username ) {
            if ( get_transient( $this->transient_name ) ) {
                $datas = get_transient( $this->transient_name );
                $datas['tried']++;

                if ( $datas['tried'] <= $this->failed_login_limit )
                    set_transient( $this->transient_name, $datas , $this->lockout_duration );
            } else {
                $datas = array(
                    'tried'     => 1
                );
                set_transient( $this->transient_name, $datas , $this->lockout_duration );
            }
        }


        /**
         * Return difference between 2 given dates
         * @param  int      $time   Date as Unix timestamp
         * @return string           Return string
         */
        private function when( $time ) {
            if ( ! $time )
                return;

            $right_now = time();

            $diff = abs( $right_now - $time );

            $second = 1;
            $minute = $second * 60;
            $hour = $minute * 60;
            $day = $hour * 24;

            if ( $diff < $minute )
                return floor( $diff / $second ) . ' secondes';

            if ( $diff < $minute * 2 )
                return "about 1 minute ago";

            if ( $diff < $hour )
                return floor( $diff / $minute ) . ' minutes';

            if ( $diff < $hour * 2 )
                return 'about 1 hour';

            return floor( $diff / $hour ) . ' hours';
        }
    }
}
//Enable
new limit_login_attempts();

It does not display any messages so I have no idea now to check if it works properly.

I've never user WP_Error before and docs were not very useful. How or where should this error output?

• I have front-end login which should not change anything because it uses autheniticate and wp_login_failed filter/action, right?
• This is the first time I heard about transients & I read that this value might not be saved in database. Checked anyway - not there.
• No errors

Why it doesn't work? Few tips about how to debug it myself would also be awesome.

Answer 1

Define as a class variable:

var $error = null;

Add it to your constructor:

$this->error = new WP_Error();

In you codition, if you have any error, add it

$error->add('login_atttempts', 'Your message!');

And you can check at the end:

$errors = $error->get_error_messages();
if( ! empty($errors)) {
   wp_die($error) // You can pass the whole WP_Error object to wp_die()
}