Skip to content

Conversation

@link2xt
Copy link
Collaborator

@link2xt link2xt commented Nov 7, 2025

This is a WIP PR on top of #7348, somehow it makes CI get stuck and timeout so I have split it off.

EDIT: the bug is that during new transport configuration transport_id 0 is used, so UID validity is set for 0 and then gets reset when reconnected and messages are skipped.

The problem can be reproduced when running on the server, tests randomly fail:

FAILED tests/test_securejoin.py::test_setup_contact_resetup - Failed: Timeout (>60.0s) from pytest-timeout.
FAILED tests/test_something.py::test_import_export_keys - Failed: Timeout (>60.0s) from pytest-timeout.
FAILED tests/test_something.py::test_delete_deltachat_folder - Failed: Timeout (>60.0s) from pytest-timeout.

But on a slower wi-fi network that has higher ping to the email server this never fails for me.

Anyway, need to fix it, then merge back into #7348

@link2xt link2xt changed the base branch from main to link2xt/multitransport November 7, 2025 00:55
@link2xt link2xt force-pushed the link2xt/multitransport-debug branch from f73cc7c to b514a50 Compare November 7, 2025 07:15
@link2xt link2xt force-pushed the link2xt/multitransport branch from 5612b91 to 2bbcdcf Compare November 7, 2025 07:15
@link2xt link2xt force-pushed the link2xt/multitransport-debug branch from b514a50 to e58a751 Compare November 8, 2025 03:28
@link2xt link2xt force-pushed the link2xt/multitransport branch 2 times, most recently from 7461a3f to 15b8cc9 Compare November 8, 2025 20:14
@link2xt link2xt force-pushed the link2xt/multitransport-debug branch from e58a751 to eba2a8f Compare November 8, 2025 20:14
@link2xt link2xt force-pushed the link2xt/multitransport branch from 15b8cc9 to 277eb64 Compare November 8, 2025 22:08
@link2xt link2xt force-pushed the link2xt/multitransport-debug branch from eba2a8f to 44c7f34 Compare November 8, 2025 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants