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

fix: Fixes download issues on cancellation, download status logs, connection stability #2686

Closed
wants to merge 11 commits into from

Conversation

islathehut
Copy link
Collaborator

Pull Request Checklist

  • I have linked this PR to a related GitHub issue.
  • I have added a description of the change (and Github issue number, if any) to the root CHANGELOG.md.

(Optional) Mobile checklist

Please ensure you completed the following checks if you did any changes to the mobile package:

  • I have run e2e tests for mobile
  • I have updated base screenshots for visual regression tests

leblowl and others added 11 commits December 20, 2024 14:15
* WIP: Update OrbitDB/IPFS/libp2p

* Various updates

* Various updates

* Fixes

* Updates

* Clean up NestJS modules and fix storage.service tests

* Fixes, mostly around restarting app

* Use custom storage to ensure we write data as Uint8array + rename OrbitdbService

* Ignore logs

* More storage related stuff

* (Maybe) fix the ping issue and mild refactor for my sanity

* Fix reconnections and update logging

* Get desktop to run without the customevent flag

* Mobile testing

* Ensure mobile can function with JS customevents

* Fix for issue with not having all blocks

* Fix common tests

* Fix almost all tests on the backend, etc.

* Fix for customevent errors in ci tests

* Fix state manager test and backend test node options

* Update class-validator

* Test mac e2e ci issue

* Fix tor binary dir issue in ci

* Try again

* Random fixes for e2e tests

* Fix file manager

* Fix code scanning alert no. 1: Insecure randomness

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

* Some digging on why the backend process isn't starting on linux

* Not sure why this didn't show up as errors in vscode

* Get fork argv

* Remove debug code and add patch for connection monitor

* Log everything

* Remove env variables as test

* Fix url scheme handler error on linux

* Remove debug logging

* Trying something

* See if we can bypass error I'm seeing on linux

* Put logs back to a sane level

* Reorder e2e test cases so we can guarantee everything has a chance to run

* Updating dependencies for mobile

* Add helia as a package

* Revert "Updating dependencies for mobile"

This reverts commit bcb9a2a.

* Some minor updates

* Fix logging and get rid of annoying malformed rn-bridge error on background

* Cleanup, fix ios resume, add volta params

* Ensure helia is bootstrapped in CI

* Fix up closing and make sure tests exit when they finish

* E2E test updates

* Merge branch 'develop' into update-orbitdb

* Fix the duplicated/unregistered bug

* Invitation link is failing consistently so do it last

* Try macos-14

* Update CSRs test since the order that orbitdb returns events in changed

* Fix jest issue

* Fix connection manager tests

* Use macos-13

* Log test name on all test cases in e2e

* Reformat and update back compat version

* Don't run back compat tests on this branch

* Add log to invitation

* Remove empty if

* Add back old xdg-settings call

* Get branch correctly on CI

* Fix vulnerable dependencies

* Do a better job - why didn't they upgrade a vulnerable package when releasing a major version?

* Check something

* Try something again

* Try this method

* Not

* Test

* Update e2e-linux.yml

* Update e2e-linux.yml

* remove print statement

* Use windows latest

* Use windows latest large

* Switch back

* Remove node_datachannel references

* Remove inaccurate comments

* Remove unnecessary changes

* Share the same constants and enums with identity

* Merge remote-tracking branch 'origin/develop' into update-orbitdb

* Update PEER_ID_REGEX to match 52-character peer IDs

* generate random device name instead of using mac

* fix connections manager test

* Fix leave community issue

* Fix common tests

* Fix backend tests

* Update snapshots

* Purge ALL data

* add extension error handling

* fix bigfiles test

* Add file upload test cases for single user

* Add file upload/download/cancel test cases for multiple users

* fix snapshots

* Generate big test file at runtime

* E2E refactor to (mostly) not use sleep statements

* Add extra visibility checks

---------

Co-authored-by: Lucas Leblow <lucas@quiet.chat>
Co-authored-by: Isla Koenigsknecht <ikoenigbu@gmail.com>
Co-authored-by: Isla Koenigsknecht <ikoenigsknecht@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Taea Vogel <taelxvie@gmail.com>
 - @quiet/desktop@3.0.0-alpha.0
 - @quiet/mobile@3.0.0-alpha.0
 - @quiet/desktop@3.0.0-alpha.1
 - @quiet/mobile@3.0.0-alpha.1
* Upgrade to orbitdb 2.4.3 (with libp2p 2.x and helia 5.x)

* Fix leave community

* Mild test fixes

* Fix file manager download test

* Fix connections manager tests

* Remove unused keys

* Undo logging and try to fix linux tests

* submodule stuff

* Go back to helia as a package

* Forgot one

* Workaround for failing test cleanup

* Add back compat skip branch

* Upgrade to node 18.20.4 to support noise as a submodule

* Fix build in CI

* Add noise submodule to get the latest code without WASM

* Reuse noise keys and fix some libp2p settings

* Update CHANGELOG.md
 - @quiet/desktop@3.0.0-alpha.2
 - @quiet/mobile@3.0.0-alpha.2
@islathehut islathehut closed this Jan 2, 2025
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