Page MenuHomePhorge
Differential D4546

Add imap:cleanup command
ClosedPublic
Actions

Authored by machniak on Sep 25 2023, 3:46 PM.
Tags
None
Referenced Files
F12010200: D4546.diff
Sat, May 4, 11:13 PM
Unknown Object (File)
Mon, Apr 29, 2:35 AM
Unknown Object (File)
Thu, Apr 18, 10:34 AM
Unknown Object (File)
Wed, Apr 17, 10:56 AM
Unknown Object (File)
Wed, Apr 17, 10:56 AM
Unknown Object (File)
Wed, Apr 17, 10:56 AM
Unknown Object (File)
Wed, Apr 17, 10:56 AM
Unknown Object (File)
Mon, Apr 15, 4:27 AM
View All 80 Files
Subscribers
mollekopf
Restricted Project

Details

Reviewers
None
Group Reviewers
Restricted Project
Commits
rKed51185b3ee1: Add imap:cleanup command
Summary

Replaces aclCleanupJob run on user delete with a command that should be run at night.

Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

machniak requested review of this revision.Sep 25 2023, 3:46 PM
machniak created this revision.
Harbormaster completed remote builds in B44278: Diff 13051.Sep 25 2023, 3:46 PM
mollekopf subscribed.Sep 26 2023, 1:24 PM
mollekopf added inline comments.
src/app/Console/Kernel.php
30

Let's first merge this without directly scheduling the command, so we can first make sure it is going to work as intended.

mollekopf added a comment.Sep 26 2023, 1:25 PM

Looks reasonable otherwise, let's merge it without the schedule and then have a look.

machniak updated this revision to Diff 13060.Sep 26 2023, 1:30 PM
  • Comment out imap:cleanup in schedule
Harbormaster completed remote builds in B44290: Diff 13060.Sep 26 2023, 1:30 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 26 2023, 1:32 PM
Closed by commit rKed51185b3ee1: Add imap:cleanup command (authored by machniak). · Explain Why
This revision was automatically updated to reflect the committed changes.
machniak added a commit: rKed51185b3ee1: Add imap:cleanup command.

Revision Contents
Changeset List

PathSize
src/
app/
Backends/
71 lines
Console/
Commands/
52 lines
3 lines
tests/
Feature/
Backends/
49 lines

Diff 13063

View Options

src/app/Backends/IMAP.php

Loading...
View Options

src/app/Console/Commands/ImapCleanupCommand.php

Loading...
View Options

src/app/Console/Kernel.php

Loading...
View Options

src/tests/Feature/Backends/IMAPTest.php

Loading...