On Jun 3, 2012, at 19:00, Matt Simerson wrote:

> removed hook_connect, unnecessary.

I think it's there so if you log or otherwise use the note, it'll be reasonably 
initialized.

> -  if ($badcmdcount >= $self->{_unrec_cmd_max}) {
> -    my $msg = "Closing connection, $badcmdcount unrecognized commands.";
> -    $self->log(LOGINFO, "fail: $msg");
> -    return (DENY_DISCONNECT, "$msg Perhaps you should read RFC 2821?");
> -  }
> +    if ( $count < $self->{_unrec_cmd_max} ) {
> +        $self->log(LOGINFO, "'$cmd', ($count)");
> +        return DECLINED;
> +    };

Why is the new version better?   (Other than just being different).



Ask

Reply via email to