Matt Simerson
Matt Simerson
It's a blocker for me too.
I'm inclined to agree that overriding badrcptto is not a great idea. If the `RCPT TO` address is listed in badrcptto, then it should **always** be blocked.
Having seen this same issue in both Qpsmtpd and Haraka, I don't think that that per-plugin timeouts are the correct answer. A more robust solution is for the MTA to...
I find the best way to illuminate the subtleties is to write some test coverage. If this PR introduced tests that exercised the code, I bet it would be better...
I'm thinking out loud here. I'm really not terribly fond of fixing one regex problem by throwing more regex at it. I think better and faster (runtime) would be to...
Even better!
My preferred solution is one that: 1. adds tests which exercise the problem in this issue 2. solves the problem 3. prevents regression (see 1.) Bonus points for replacing custom...
> Do you mean this solution would be OK if I add unit test Exactly. > there are no test at all for uribl for now, so I'm not sure...
@dani, did you make progress on using Data::Validate::Domain? If so, can you push that to this branch?
Hello @bunkobugsy. I read through the bug tracker link and it didn't seem conclusive to me. If I was sure about removing line 199, I would have. The fact that...