Paths

Table of Contentst

Nicely handle "duplicate entry" errors in greylisting
ClosedPublic
Actions

Authored by machniak on Mon, May 18, 12:49 PM.

Project Tags

None

Referenced Files

	F125449059: D5895.1779475049.diff
	Thu, May 21, 6:37 PM

	F125437484: D5895.1779471589.diff
	Thu, May 21, 5:39 PM

	F125287963: D5895.1779404206.diff
	Wed, May 20, 10:56 PM

Subscribers

mollekopf

Restricted Project

Details

Reviewers

mollekopf

Group Reviewers

Restricted Project

Commits

rKafd80adc1afb: Nicely handle "duplicate entry" errors in greylisting

Summary

There's a race condition between two concurent greylist policy checks
for the same recipient+sender. Handle it nicely.

Test Plan

./phpunit

Diff Detail

Repository

rK kolab

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

machniak requested review of this revision.Mon, May 18, 12:49 PM

machniak created this revision.

Harbormaster completed remote builds in B57524: Diff 16856.Mon, May 18, 12:50 PM

mollekopf accepted this revision.Tue, May 19, 3:15 PM

mollekopf subscribed.

mollekopf added inline comments.

src/app/Policy/Greylist.php
235	If we're not using it we shouldn't try to update it.

This revision is now accepted and ready to land.Tue, May 19, 3:15 PM

Closed by commit rKafd80adc1afb: Nicely handle "duplicate entry" errors in greylisting. · Explain WhyThu, May 21, 10:47 AM

This revision was automatically updated to reflect the committed changes.

machniak added a commit: rKafd80adc1afb: Nicely handle "duplicate entry" errors in greylisting.

Revision Contents
Changeset List

Path

Size

src/

app/

Policy/

Greylist.php

lines

Diff 16873

View Options

src/app/Policy/Greylist.php

Nicely handle "duplicate entry" errors in greylistingClosedPublicActions