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

Encrypt large files with temporary file #11777

Closed
wants to merge 2 commits into from
Closed

Conversation

tobiasKaminsky
Copy link
Member

@tobiasKaminsky tobiasKaminsky commented Jun 27, 2023

  • Tests written, or not not needed

@github-actions
Copy link

Codacy

Lint

TypemasterPR
Warnings7676
Errors00

SpotBugs

CategoryBaseNew
Bad practice2628
Correctness8383
Dodgy code435440
Experimental21
Internationalization99
Malicious code vulnerability22
Multithreaded correctness99
Performance5658
Security1818
Total640648

SpotBugs increased!

long to support more than 2gb files

Signed-off-by: tobiasKaminsky <[email protected]>
@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/11777.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions
Copy link

blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed.

surinder-tsys added a commit to nextmcloud/android that referenced this pull request Aug 28, 2023
@msviba msviba mentioned this pull request Sep 23, 2023
17 tasks
surinder-tsys added a commit to nextmcloud/android that referenced this pull request Oct 20, 2023
@alperozturk96
Copy link
Collaborator

@tobiasKaminsky This PR not valid anymore. New PR

@AndyScherzinger AndyScherzinger requested review from ZetaTom, AlvaroBrey and alperozturk96 and removed request for AlvaroBrey April 7, 2024 15:44
@alperozturk96
Copy link
Collaborator

alperozturk96 commented Apr 9, 2024

@AndyScherzinger @tobiasKaminsky @ZetaTom We should close this PR not valid anymore.

@alperozturk96
Copy link
Collaborator

Duplication of this PR

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.

2 participants