...and at least include the date as an indicator which message triggered
the issue (we don't just have an identifier available), and log the
error message.

Diff Detail

Repository

rK kolab

Lint

Lint Skipped

Unit

No Test Coverage

Build Status

Buildable 52992
Build 18786: arc lint + arc unit

Event Timeline

mollekopf requested review of this revision.Mar 19 2025, 4:09 PM

mollekopf created this revision.

Harbormaster completed remote builds in B52983: Diff 14763.Mar 19 2025, 4:09 PM

mollekopf added a reviewer: Restricted Project.Mar 19 2025, 4:11 PM

machniak requested changes to this revision.Mar 19 2025, 5:40 PM

machniak subscribed.

machniak added inline comments.

src/app/DataMigrator/Driver/IMAP.php
157	Is there a problem with `$item->id`?
160	I'm not sure if `\n` in a log message is good idea.

This revision now requires changes to proceed.Mar 19 2025, 5:40 PM

Adressed comments

Harbormaster completed remote builds in B52992: Diff 14772.Mar 20 2025, 7:28 AM

mollekopf added inline comments.Mar 20 2025, 7:28 AM

src/app/DataMigrator/Driver/IMAP.php
157	Nope, I just thought that the id was in data from looking at the item construction
160	Good for console output, bad for log processing. Removed it for now.