Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Application throws error if it's closed during heavy replication #463

Closed
EmiM opened this issue Apr 21, 2022 · 7 comments
Closed

Application throws error if it's closed during heavy replication #463

EmiM opened this issue Apr 21, 2022 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@EmiM
Copy link
Contributor

EmiM commented Apr 21, 2022

image

2022-04-21T17:26:52.033Z waggle:db Stopping OrbitDB
2022-04-21T17:26:52.090Z waggle:db Stopping IPFS
TypeError: Cannot read properties of null (reading 'length')
    at Replicator.onReplicationProgress (C:\Users\Emi\AppData\Local\Programs\quiet\resources\app.asar\node_modules\@quiet\waggle\node_modules\orbit-db-store\src\Store.js:98:25)
    at onProgressCallback (C:\Users\Emi\AppData\Local\Programs\quiet\resources\app.asar\node_modules\@quiet\waggle\node_modules\orbit-db-store\src\Replicator.js:147:14)
    at addToResults (C:\Users\Emi\AppData\Local\Programs\quiet\resources\app.asar\node_modules\@quiet\waggle\node_modules\ipfs-log\src\entry-io.js:116:17)
    at C:\Users\Emi\AppData\Local\Programs\quiet\resources\app.asar\node_modules\@quiet\waggle\node_modules\ipfs-log\src\entry-io.js:151:11
C:\Users\Emi\AppData\Local\Programs\quiet\resources\app.asar\node_modules\@quiet\waggle\lib\libp2p\connectionsManager.js:224
            throw new Error();
            ^

Error
    at process.<anonymous> (C:\Users\Emi\AppData\Local\Programs\quiet\resources\app.asar\node_modules\@quiet\waggle\lib\libp2p\connectionsManager.js:224:19)
    at process.emit (node:events:390:28)
    at processEmit [as emit] (C:\Users\Emi\AppData\Local\Programs\quiet\resources\app.asar\node_modules\@quiet\waggle\node_modules\signal-exit\index.js:197:34)
    at emit (node:internal/process/promises:136:22)
    at processPromiseRejections (node:internal/process/promises:242:25)
    at processTicksAndRejections (node:internal/process/task_queues:97:32)
@holmesworcester holmesworcester added the bug Something isn't working label Apr 22, 2022
@holmesworcester
Copy link
Contributor

should we assume this is fixed in the latest release and QA it?

@siepra siepra self-assigned this May 2, 2022
@siepra
Copy link
Contributor

siepra commented May 2, 2022

should we assume this is fixed in the latest release and QA it?

In the logs above we can see the problem lies in networking layer so I'm afraid the answer is no. I'll try to reproduce it and get to the bottom of the problem

@siepra
Copy link
Contributor

siepra commented May 2, 2022

So here are the logs from closing window. It seems like the OrbitDB has stopped but waggle still tries to accomplish queued replication tasks (which causes error on reading undefined oplog instance)

Zrzut ekranu 2022-05-2 o 11 35 31

@siepra
Copy link
Contributor

siepra commented May 2, 2022

It's OrbitDB problem. I opened a ticket and prepared a sample repo that illustrates the issue
orbitdb-archive/orbit-db-store#121

@siepra siepra removed their assignment May 9, 2022
@siepra
Copy link
Contributor

siepra commented May 9, 2022

Patched with version 0.28.5 (orbitdb)

@siepra siepra self-assigned this May 9, 2022
@siepra
Copy link
Contributor

siepra commented May 10, 2022

This issue is also related as it resolves problems with performance regressions #491

@siepra
Copy link
Contributor

siepra commented May 10, 2022

Related PR #490

@kingalg kingalg closed this as completed Jun 21, 2022
@holmesworcester holmesworcester moved this to Done in Quiet Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

4 participants