Page MenuHomePhorge

Fix greylisting setting
ClosedPublic

Authored by machniak on Aug 25 2021, 12:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 18, 4:47 AM
Unknown Object (File)
Tue, Apr 30, 10:56 AM
Unknown Object (File)
Sat, Apr 27, 6:32 AM
Unknown Object (File)
Fri, Apr 26, 7:52 PM
Unknown Object (File)
Fri, Apr 26, 1:52 PM
Unknown Object (File)
Thu, Apr 25, 2:49 PM
Unknown Object (File)
Tue, Apr 23, 8:31 PM
Unknown Object (File)
Apr 20 2024, 11:35 AM
Subscribers
Restricted Project

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rKa53ecb760738: Fix greylisting setting
Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Branch
dev/greylist-tests
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 36391
Build 14407: arc lint + arc unit

Event Timeline

machniak created this revision.
  • Small perf. improvement, skip redundant query
  • Performance optimizations
  • Move greylisting setting back to user_settings, cleanup
  • Remove redundant ->limit(1)
mollekopf added inline comments.
src/app/Policy/Greylist/Request.php
150

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
27 ↗(On Diff #8086)

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.

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.

This revision is now accepted and ready to land.Aug 27 2021, 9:20 AM
This revision was automatically updated to reflect the committed changes.