Page MenuHomePhorge
Differential D4692

Improve greylist handler
ClosedPublic
Actions

Authored by machniak on Apr 23 2024, 5:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 5, 9:40 PM
Unknown Object (File)
Fri, Aug 30, 9:05 AM
Unknown Object (File)
Fri, Aug 30, 8:34 AM
Unknown Object (File)
Fri, Aug 30, 8:34 AM
Unknown Object (File)
Thu, Aug 29, 12:25 PM
Unknown Object (File)
Thu, Aug 29, 7:06 AM
Unknown Object (File)
Wed, Aug 28, 1:15 PM
Unknown Object (File)
Fri, Aug 23, 8:02 PM
View All 63 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 13530

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...