Details

src/app/Policy/Greylist/Request.php
113	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?
src/database/migrations/2021_08_25_120000_drop_greylist_settings_table.php
28	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.

In D2815#33931, @mollekopf wrote:

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.

mollekopf accepted this revision.Aug 27 2021, 9:20 AM

This revision is now accepted and ready to land.Aug 27 2021, 9:20 AM

Closed by commit rKa53ecb760738: Fix greylisting setting (authored by machniak). · Explain WhyAug 27 2021, 9:23 AM

This revision was automatically updated to reflect the committed changes.

Fix greylisting setting
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 8137

src/app/Http/Controllers/API/V4/PolicyController.php

src/app/IP4Net.php

src/app/IP6Net.php

src/app/Policy/Greylist/Request.php

src/app/Policy/Greylist/Setting.php

src/app/Traits/UserConfigTrait.php

src/app/Utils.php

src/database/migrations/2021_08_25_120000_drop_greylist_settings_table.php

src/resources/vue/Admin/User.vue

src/resources/vue/User/Info.vue

src/tests/Browser/Reseller/UserTest.php

src/tests/Feature/Controller/UsersTest.php

src/tests/Feature/Stories/GreylistTest.php

src/tests/Feature/UserTest.php

Fix greylisting settingClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 8137

src/app/Http/Controllers/API/V4/PolicyController.php

src/app/IP4Net.php

src/app/IP6Net.php

src/app/Policy/Greylist/Request.php

src/app/Policy/Greylist/Setting.php

src/app/Traits/UserConfigTrait.php

src/app/Utils.php

src/database/migrations/2021_08_25_120000_drop_greylist_settings_table.php

src/resources/vue/Admin/User.vue

src/resources/vue/User/Info.vue

src/tests/Browser/Reseller/UserTest.php

src/tests/Feature/Controller/UsersTest.php

src/tests/Feature/Stories/GreylistTest.php

src/tests/Feature/UserTest.php

Fix greylisting setting
ClosedPublic
Actions

Revision Contents
Changeset List