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