Page MenuHomePhorge
Differential D4692

Improve greylist handler
ClosedPublic
Actions

Authored by machniak on Apr 23 2024, 5:24 PM.
Tags
None
Referenced Files
F16727661: D4692.diff
Mon, Nov 4, 8:37 PM
Unknown Object (File)
Sat, Nov 2, 9:45 PM
Unknown Object (File)
Tue, Oct 29, 11:53 AM
Unknown Object (File)
Sun, Oct 27, 1:44 AM
Unknown Object (File)
Sat, Oct 26, 7:28 AM
Unknown Object (File)
Mon, Oct 21, 2:54 AM
Unknown Object (File)
Thu, Oct 17, 4:12 AM
Unknown Object (File)
Tue, Oct 8, 9:02 PM
View All 74 Files
Subscribers
Restricted Project

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rK9e94c8a1533a: Improve greylist handler
Summary
  • introduce DB transaction
  • limit number of selects from greylist_connect table
  • limit number of whitelist record updates
Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Branch
dev/spf-perf
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 47148
Build 17967: arc lint + arc unit

Event Timeline

machniak requested review of this revision.Apr 23 2024, 5:24 PM
machniak created this revision.
Harbormaster completed remote builds in B47142: Diff 13503.Apr 23 2024, 5:24 PM
machniak updated this revision to Diff 13506.Apr 24 2024, 2:27 PM
  • Improve single sql query and coding style
Harbormaster completed remote builds in B47145: Diff 13506.Apr 24 2024, 2:27 PM
machniak updated this revision to Diff 13509.Apr 24 2024, 3:53 PM
  • Move greylist policy controller tests to more appropriate place
  • Speed up greylist tests
Harbormaster completed remote builds in B47148: Diff 13509.Apr 24 2024, 3:53 PM
mollekopf accepted this revision.May 17 2024, 11:53 AM
This revision is now accepted and ready to land.May 17 2024, 11:53 AM
Closed by commit rK9e94c8a1533a: Improve greylist handler (authored by machniak). · Explain WhyMay 20 2024, 11:09 AM
This revision was automatically updated to reflect the committed changes.
machniak added a commit: rK9e94c8a1533a: Improve greylist handler.

Revision Contents
Changeset List

PathSize
src/
app/
Listeners/
2 lines
Policy/
Greylist/
177 lines
tests/
Feature/
Controller/
87 lines
Stories/
51 lines

Diff 13509

View Options

src/app/Listeners/SqlDebug.php

Loading...
View Options

src/app/Policy/Greylist/Request.php

Loading...
View Options

src/tests/Feature/Controller/PolicyTest.php

Loading...
View Options

src/tests/Feature/Stories/GreylistTest.php

Loading...