Skip to content

Conversation

GretaD
Copy link
Contributor

@GretaD GretaD commented Sep 8, 2025

fixes #11024

Another approach would be if we handle initialization in sync() instead. Smth like:

if (init) { await wait(5000) await this.loadEnvelopes() }

Signed-off-by: greta <gretadoci@gmail.com>
Signed-off-by: greta <gretadoci@gmail.com>
@GretaD GretaD marked this pull request as ready for review September 8, 2025 14:58
@GretaD
Copy link
Contributor Author

GretaD commented Sep 8, 2025

please test

@ChristophWurst
Copy link
Member

What caused the endless loop before?

@GretaD
Copy link
Contributor Author

GretaD commented Sep 9, 2025

What caused the endless loop before?

Check the async loadEnvelopes(). That’s a recursive call, loadEnvelopes() calls itself again and again after waiting 15 seconds.

@ChristophWurst
Copy link
Member

Seems intentional?

// Keep trying

@GretaD
Copy link
Contributor Author

GretaD commented Sep 9, 2025

Seems intentional?

// Keep trying

to keep trying, yes probably. To keep trying forever, i doubt its intentional. Do you think this approach is not the solution we want?

@ChristophWurst
Copy link
Member

The idea is to wait until the initial syncing is done. This should happen eventually.

@ChristophWurst
Copy link
Member

Do you have an alternative for retrying until it is no longer locked? Retry should definitely stop for any other error

Signed-off-by: greta <gretadoci@gmail.com>
@GretaD GretaD marked this pull request as draft September 19, 2025 12:56
Signed-off-by: greta <gretadoci@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Opening a mailbox that isn't cached causes an endless burst of requests

2 participants