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

feat(2744): Encrypt files in IPFS #2748

Merged
merged 14 commits into from
Feb 17, 2025
Merged

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

@islathehut islathehut linked an issue Feb 14, 2025 that may be closed by this pull request
@islathehut islathehut requested a review from adrastaea February 14, 2025 19:08
@adrastaea
Copy link
Collaborator

Failing ipfs test on my machine

  ● IpfsFileManagerService › file uploaded to IPFS then can be downloaded
    expect(jest.fn()).toHaveBeenNthCalledWith(n, ...expected)
    n: 7
    Expected: "messageMediaUpdated", ObjectContaining {"cid": StringContaining "bafy", "enc": {"header": Any<String>, "recipient": {"generation": 0, "name": "member", "type": "ROLE"}}, "ext": ".png", "height": 44, "message": {"channelId": "channelId", "id": "id"}, "name": "test-image", "path": StringContaining ".png", "size": 15881, "width": 824}
    Received
           4: "messageMediaUpdated", {"cid": "bafybeiam4pgnciac6qme4cepvzjdejgtznu2ljopiokbdjyfk53fsf66om", "enc": {"header": "JDMXg8VH27T0zY43jxUnLXriaB1yk3ze", "recipient": {"generation": 0, "name": "member", "type": "ROLE"}}, "ext": ".png", "height": 44, "message": {"channelId": "channelId", "id": "id"}, "name": "test-image", "path": "/var/folders/n1/h78wyh1x3rb8qlcxn391_3_00000gn/T/quietTestTmp_-71247-EzmlslutfHJ7/.quiet-test-nest-backend/uploads/1739817888875_c13db76935ff9d83f0269fc8c6185235_test-image.png", "size": 15881, "tmpPath": undefined, "width": 824}
    ->     7: "downloadProgress", {"cid": "bafybeiam4pgnciac6qme4cepvzjdejgtznu2ljopiokbdjyfk53fsf66om", "downloadProgress": {"downloaded": 15881, "size": 15881, "transferSpeed": 0}, "downloadState": "canceled", "mid": "id"}
    Number of calls: 7

@adrastaea adrastaea self-requested a review February 17, 2025 19:53
@islathehut islathehut merged commit 13e231a into develop Feb 17, 2025
26 of 30 checks passed
@islathehut islathehut deleted the feat/2744-file-encryption branch February 17, 2025 20:28
islathehut added a commit that referenced this pull request Feb 19, 2025
* Update auth

* Add stream encryption to crypto service

* Add oneToOne chunker

* Add file encryption

* Fix (most) backend tests

* Update .gitignore

* Log at start of timer, too

* Tweak logs

* Update branch on submodule

* Update action.yml

* Update ipfs-file-manager.service.spec.ts

* Fix test

* Point auth at main

* Less spammy logs
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.

File encryption
2 participants