- Group Reviewers
- rKa53ecb760738: Fix greylisting setting
- rK kolab
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
I would say if there is no recipient set $enabled = false so we bail out after capturing the sender statistics?
@vanmeeuwen Should we capture sender statistics if the recipient is not found,? should we greylist if the recipient is not found?
I suppose in principle we have to (we'd be running code expecting the table).
Looks good to me. We should probably also create a policycontrollertest, to test if this actually works, but that doesn't need to be in this diff.
There are some controller tests in tests/Feature/Stories, but I agree not enough of the controller alone.