Page MenuHome GnuPG
Create Task
Maniphest T7535

Kleopatra: "Decrypt/Verify all files in folder" doesn't work
Testing, LowPublic
Actions

Description

How to reproduce:

  • Right-click a folder in Dolphin (or some other suitable file browser). So that the test makes sense the folder should contain at least one (detached) signed or encrypted file.
  • Select "Decrypt/Verify All Files In Folder".

Expected result:
Kleopatra does what I asked for.

Actual result:
Kleopatra shows an error message:

Failed to find encrypted or signed data in one or more files.
If they contain signed or encrypted data please report a bug (see Help->Report Bug).

I have not checked if this also happens on Windows.

Details

Version
Version 4.0.0.250370+git20250218T171516~c55ebbc5a (25.03.70)

Revisions and Commits

rKLEOPATRA Kleopatra
rKLEOPATRA5968653af605 Remove "Decrypt/Verify all files in folder"

Event Timeline

• ikloecker created this task.Tue, Feb 18, 7:18 PM
• TobiasFella added a subscriber: • TobiasFella.Wed, Feb 19, 11:26 AM

We don't have this exact action on windows, but the normal "Decrypt & Verify" action shows up for folders there (and doesn't work either).

FWIW, i can't find code to support this feature (iterating over the folder, selecting relevant files, .... somewhere in decryptverifyfilescontroller.cpp?) in kleopatra at all. The git history also doesn't lead me anywhere useful. I wonder if this ever worked?

• werner added a subscriber: • werner.Wed, Feb 19, 12:05 PM

I can't remember that we ever had support this. It is also not easy to come up with the good way to present the status for all files in a folder. We would need to define a format similar to what sha1sum uses: A list of file with they signature file or so. Note that kleopatra has support for running sha256sum in such a way.

• werner triaged this task as Low priority.Wed, Feb 19, 12:06 PM
• werner edited projects, added Feature Request; removed Bug Report.
• ikloecker added a comment.Wed, Feb 19, 1:22 PM

We do support "Decrypt & Verify" for multiple files (including the presentation of the status) so that it would be easy to do the same for all files in a folder (question is if this should even be recursive). Digging into the history I found that the desktop file was added shortly before Kleopatra 2.0.0-rc1, but that there wasn't any code for iterating a folder, i.e. this can never have worked.

I suggest to remove the desktop file unless we have a good use case. People can always select all files in a folder and Decrypt & Verify those files.

• TobiasFella added a commit: rKLEOPATRA5968653af605: Remove "Decrypt/Verify all files in folder".Wed, Feb 19, 2:29 PM
• TobiasFella added a comment.Wed, Feb 19, 3:46 PM

Removed in https://invent.kde.org/pim/kleopatra/-/merge_requests/369

• TobiasFella changed the task status from Open to Testing.Wed, Feb 19, 3:46 PM
• TobiasFella claimed this task.