Page MenuHomePhorge
Differential D4692

Improve greylist handler
ClosedPublic
Actions

Authored by machniak on Apr 23 2024, 5:24 PM.
Tags
None
Referenced Files
F12488451: D4692.id13503.diff
Sun, Jun 2, 12:33 AM
F12485241: D4692.id13509.diff
Sat, Jun 1, 6:46 PM
F12477351: D4692.id.diff
Sat, Jun 1, 6:35 AM
Unknown Object (File)
Tue, May 28, 6:50 PM
Unknown Object (File)
Mon, May 27, 7:23 PM
Unknown Object (File)
Mon, May 27, 6:29 PM
Unknown Object (File)
Mon, May 27, 9:46 AM
Unknown Object (File)
Fri, May 24, 12:19 PM
View All 36 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.Fri, May 17, 11:53 AM
This revision is now accepted and ready to land.Fri, May 17, 11:53 AM
Closed by commit rK9e94c8a1533a: Improve greylist handler (authored by machniak). · Explain WhyMon, May 20, 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...