wip

Signed-off-by: tobiasKaminsky <[email protected]>
nextcloud · Aug 30, 2023 · 63b32a8 · 63b32a8
1 parent bda0a2a
commit 63b32a8
Show file tree

Hide file tree

Showing 8 changed files with 171 additions and 84 deletions.
diff --git a/app/src/main/java/com/owncloud/android/operations/UnshareOperation.java b/app/src/main/java/com/owncloud/android/operations/UnshareOperation.java
@@ -79,7 +79,7 @@ protected RemoteOperationResult run(OwnCloudClient client) {
         if (share != null) {
             OCFile file = getStorageManager().getFileByEncryptedRemotePath(remotePath);
 
-            if (file.isEncrypted()) {
+            if (file.isEncrypted() && share.getShareType() != ShareType.PUBLIC_LINK) {
                 // E2E: lock folder
                 try {
                     token = EncryptionUtils.lockFolder(file, client, file.getE2eCounter() + 1);
@@ -128,9 +128,11 @@ protected RemoteOperationResult run(OwnCloudClient client) {
 
             if (result.isSuccess()) {
                 // E2E: unlock folder
-                RemoteOperationResult<Void> unlockResult = EncryptionUtils.unlockFolder(file, client, token);
-                if (!unlockResult.isSuccess()) {
-                    return new RemoteOperationResult<>(new RuntimeException("Unlock failed"));
+                if (file.isEncrypted() && share.getShareType() != ShareType.PUBLIC_LINK) {
+                    RemoteOperationResult<Void> unlockResult = EncryptionUtils.unlockFolder(file, client, token);
+                    if (!unlockResult.isSuccess()) {
+                        return new RemoteOperationResult<>(new RuntimeException("Unlock failed"));
+                    }
                 }
 
                 Log_OC.d(TAG, "Share id = " + share.getRemoteId() + " deleted");

diff --git a/app/src/main/java/com/owncloud/android/operations/UploadFileOperation.java b/app/src/main/java/com/owncloud/android/operations/UploadFileOperation.java
diff --git a/app/src/main/java/com/owncloud/android/ui/activity/SettingsActivity.java b/app/src/main/java/com/owncloud/android/ui/activity/SettingsActivity.java
@@ -474,7 +474,7 @@ private void setupE2EKeysExist(PreferenceCategory preferenceCategoryMore) {
     }
 
     private void setupE2EMnemonicPreference(PreferenceCategory preferenceCategoryMore) {
-        String mnemonic = arbitraryDataProvider.getValue(user.getAccountName(), EncryptionUtils.MNEMONIC);
+        String mnemonic = arbitraryDataProvider.getValue(user.getAccountName(), EncryptionUtils.MNEMONIC).trim();
 
         Preference pMnemonic = findPreference("mnemonic");
         if (pMnemonic != null) {
@@ -991,7 +991,7 @@ public void handleMnemonicRequest(Intent data) {
                 RequestCredentialsActivity.KEY_CHECK_RESULT_TRUE) {
 
                 ArbitraryDataProvider arbitraryDataProvider = new ArbitraryDataProviderImpl(this);
-                String mnemonic = arbitraryDataProvider.getValue(user.getAccountName(), EncryptionUtils.MNEMONIC);
+                String mnemonic = arbitraryDataProvider.getValue(user.getAccountName(), EncryptionUtils.MNEMONIC).trim();
 
                 AlertDialog.Builder builder = new AlertDialog.Builder(this, R.style.FallbackTheming_Dialog);
                 AlertDialog alertDialog = builder.setTitle(R.string.prefs_e2e_mnemonic)

diff --git a/app/src/main/java/com/owncloud/android/ui/dialog/SetupEncryptionDialogFragment.java b/app/src/main/java/com/owncloud/android/ui/dialog/SetupEncryptionDialogFragment.java
@@ -201,7 +201,8 @@ private Dialog createDialog(View v) {
                                                                                            mnemonic);
 
                             arbitraryDataProvider.storeOrUpdateKeyValue(user.getAccountName(),
-                                                                        EncryptionUtils.PRIVATE_KEY, decryptedPrivateKey);
+                                                                        EncryptionUtils.PRIVATE_KEY,
+                                                                        decryptedPrivateKey);
 
                             dialog1.dismiss();
                             Log_OC.d(TAG, "Private key successfully decrypted and stored");

diff --git a/app/src/main/java/com/owncloud/android/ui/fragment/FileDetailSharingFragment.java b/app/src/main/java/com/owncloud/android/ui/fragment/FileDetailSharingFragment.java
@@ -219,7 +219,18 @@ private void setupView() {
 
         if (file.canReshare()) {
             if (file.isEncrypted() || (parentFile != null && parentFile.isEncrypted())) {
-                binding.searchView.setQueryHint(getResources().getString(R.string.secure_share_search));
+                if (file.getE2eCounter() == -1) {
+                    // V1 cannot share
+                    binding.searchContainer.setVisibility(View.GONE);
+                } else {
+                    binding.searchView.setQueryHint(getResources().getString(R.string.secure_share_search));
+
+                    if (file.isSharedViaLink()) {
+                        binding.searchView.setQueryHint(getResources().getString(R.string.share_not_allowed_when_file_drop));
+                        binding.searchView.setInputType(InputType.TYPE_NULL);
+                        disableSearchView(binding.searchView);
+                    }
+                }
             } else {
                 binding.searchView.setQueryHint(getResources().getString(R.string.share_search));
             }
@@ -429,6 +440,7 @@ public void refreshCapabilitiesFromDB() {
      * before reading database.
      */
     public void refreshSharesFromDB() {
+        file = fileDataStorageManager.getFileById(file.getFileId());
         ShareeListAdapter adapter = (ShareeListAdapter) binding.sharesList.getAdapter();
 
         if (adapter == null) {

diff --git a/app/src/main/java/com/owncloud/android/utils/EncryptionUtils.java b/app/src/main/java/com/owncloud/android/utils/EncryptionUtils.java
@@ -55,6 +55,7 @@
 import com.owncloud.android.lib.resources.e2ee.StoreMetadataRemoteOperation;
 import com.owncloud.android.lib.resources.e2ee.StoreMetadataV2RemoteOperation;
 import com.owncloud.android.lib.resources.e2ee.UnlockFileRemoteOperation;
+import com.owncloud.android.lib.resources.e2ee.UnlockFileV1RemoteOperation;
 import com.owncloud.android.lib.resources.e2ee.UpdateMetadataRemoteOperation;
 import com.owncloud.android.lib.resources.e2ee.UpdateMetadataV2RemoteOperation;
 import com.owncloud.android.lib.resources.status.E2EVersion;
@@ -234,7 +235,7 @@ public static EncryptedFolderMetadataFileV1 encryptFolderMetadata(
         }
 
         // set checksum
-        String mnemonic = arbitraryDataProvider.getValue(user.getAccountName(), EncryptionUtils.MNEMONIC);
+        String mnemonic = arbitraryDataProvider.getValue(user.getAccountName(), EncryptionUtils.MNEMONIC).trim();
         String checksum = EncryptionUtils.generateChecksum(decryptedFolderMetadata, mnemonic);
         encryptedFolderMetadata.getMetadata().setChecksum(checksum);
 
@@ -1421,6 +1422,14 @@ public static RemoteOperationResult<Void> unlockFolder(OCFile parentFolder, OwnC
         }
     }
 
+    public static RemoteOperationResult<Void> unlockFolderV1(OCFile parentFolder, OwnCloudClient client, String token) {
+        if (token != null) {
+            return new UnlockFileV1RemoteOperation(parentFolder.getLocalId(), token).execute(client);
+        } else {
+            return new RemoteOperationResult<>(new Exception("No token available"));
+        }
+    }
+
     public static X509Certificate convertCertFromString(String string) throws CertificateException {
         String trimmedCert = string.replace("-----BEGIN CERTIFICATE-----\n", "")
             .replace("-----END CERTIFICATE-----\n", "");

diff --git a/app/src/main/java/com/owncloud/android/utils/EncryptionUtilsV2.kt b/app/src/main/java/com/owncloud/android/utils/EncryptionUtilsV2.kt
@@ -377,8 +377,6 @@ class EncryptionUtilsV2 {
         ocFile.setE2eCounter(metadataFile.metadata.counter)
         fileDataStorageManager.saveFile(ocFile)
 
-        // TODO change metadata key always?
-
         return metadataFile
     }
 

diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -1102,4 +1102,5 @@
     <string name="ecosystem_apps_display_more">More</string>
     <string name="secure_sharing_failed">Secure sharing failed</string>
     <string name="secure_share_not_set_up">Secure sharing is not set up for this user</string>
+    <string name="share_not_allowed_when_file_drop">Resharing is not allowed during secure file drop</string>
 </resources>