Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

add test for broken conversationsdb #93

Open
elliefm opened this issue Aug 21, 2020 · 1 comment
Open

add test for broken conversationsdb #93

elliefm opened this issue Aug 21, 2020 · 1 comment

Comments

@elliefm
Copy link
Contributor

elliefm commented Aug 21, 2020

bron:

I ran fixaccount for user with the issue, so we don't have a testcase as such for this. It looks like you need a broken conversations.db for the error to come up. The specific brokenness is "mailbox got deleted, but a record for it didn't get removed from conversations.db for some reason" - so there's a G record with a folder number which is a broken pointer.

ellie:

Could we reproduce this from Cassandane by just having cyr_dbtool inject a bad G record into the cassandane account's conversations.db? Or does the bad record need to be bad in a particular way that's hard to anticipate?

bron:

I think we could - either by injecting a bad record, or by messing with $FOLDER_NAMES.

Putting this here so we don't completely forget it forever

@elliefm
Copy link
Contributor Author

elliefm commented Aug 21, 2020

Forgot some context: the crash this caused is already fixed by cyrusimap/cyrus-imapd@63c6caf, but we don't have a regression test for it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant