Forums

The WordPress MU forums have moved to the MultiSite forum on WordPress.org

Prevent registration spam from certain emails (6 posts)

  1. modifiedcontent
    Member
    Posted 2 years ago #

    I'd like to block all *.info email addresses from registering. That should be possible by hacking this function in wpmu-functions.php:

    function is_email_address_unsafe( $user_email ) {
    	$banned_names = get_site_option( "banned_email_domains" );
    	if ( is_array( $banned_names ) && empty( $banned_names ) == false ) {
    		$email_domain = strtolower( substr( $user_email, 1 + strpos( $user_email, '@' ) ) );
    		foreach( (array) $banned_names as $banned_domain ) {
    			if( $banned_domain == '' )
    				continue;
    			if (
    				strstr( $email_domain, $banned_domain ) ||
    				(
    					strstr( $banned_domain, '/' ) &&
    					preg_match( $banned_domain, $email_domain )
    				)
    			)
    			return true;
    		}
    	}
    	return false;
    }

    According to this instruction it's easy:

    Look at is_email_address_unsafe() in wpmu- functions.php - there's a preg_match() there if you have a regex in the list of domains enclosed in "/" characters. For example, /.*\.info/ will match *.info

    But I can't put it together. What am I supposed to put where? Like this?:

    strstr( $banned_domain, '/.*\.info/' )

    What if I want to ban a whole list of domains?

    Apparently the Banned Email List in wpmu admin options does not work because of a bug. Has that been fixed in latest versions? Will it be?

  2. tmoorewp
    Member
    Posted 2 years ago #

    The banned email list works just fine to block registration. Have you tried using it or are you just taking that one blog entry as saying it doesn't work?

  3. modifiedcontent
    Member
    Posted 2 years ago #

    So darcynorman is wrong?

    Does the Banned Email list accept wildcards, like *.info?

    I'll try it...

  4. kgraeme
    Member
    Posted 2 years ago #

    The darcynorman fix was included on the 30th and should be part of the 2.8.5.2 core. However, it is still doing full domains, not wildcards.

  5. modifiedcontent
    Member
    Posted 2 years ago #

    Why not wildcards? Full domains is pretty useless as spam fighting method.

  6. andrea_r
    Moderator
    Posted 2 years ago #

    Feel free to code that bit up and submit it to trac then. :)

Topic Closed

This topic has been closed to new replies.

About this Topic

Tags

No tags yet.