Page MenuHomePhorge
Differential D2728

Set errors if something fails during append
Needs RevisionPublic
Actions

Authored by mollekopf on Aug 16 2021, 4:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 18 2023, 1:14 AM
Unknown Object (File)
Oct 14 2023, 11:42 AM
Unknown Object (File)
Oct 5 2023, 6:27 AM
Unknown Object (File)
Sep 30 2023, 7:41 PM
Unknown Object (File)
Sep 29 2023, 8:35 AM
Unknown Object (File)
Sep 27 2023, 7:12 PM
Unknown Object (File)
Sep 19 2023, 4:02 PM
Unknown Object (File)
Sep 18 2023, 11:14 AM
View All 9 Files
Subscribers
machniak

Details

Reviewers
machniak
Group Reviewers
Restricted Project
Commits
rKc8a9d1936cc4: Set errors if something fails during append

Diff Detail

Repository
rK kolab
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 36079
Build 14173: arc lint + arc unit

Event Timeline

mollekopf requested review of this revision.Aug 16 2021, 4:16 PM
mollekopf created this revision.
Harbormaster completed remote builds in B36079: Diff 7768.Aug 16 2021, 4:16 PM
mollekopf added a reviewer: Restricted Project.Aug 18 2021, 10:37 AM

I added this while using the migration code because sometimes we would just silently fail.

machniak subscribed.Sep 20 2022, 1:00 PM

This is upstream code, so we should be careful, and then create a PR for Roundcube to include the changes.

src/include/rcube_imap_generic.php
3053

The message should be more explanatory, maybe "Failed to write to socket on APPEND".

3061

Same here.

machniak requested changes to this revision.Sep 20 2022, 1:00 PM
This revision now requires changes to proceed.Sep 20 2022, 1:00 PM

Revision Contents
Changeset List

PathSize
src/
include/
3 lines

Diff 7768

View Options

src/include/rcube_imap_generic.php

Loading...