Closed Bug 1725447 Opened 3 years ago Closed 3 years ago

Attachment area not fully removed when removing GPG key

Categories

(Thunderbird :: Message Compose Window, defect)

Thunderbird 91
x86_64
Linux
defect

Tracking

(thunderbird_esr91+ fixed, thunderbird92+ fixed)

RESOLVED FIXED
93 Branch
Tracking Status
thunderbird_esr91 + fixed
thunderbird92 + fixed

People

(Reporter: freaktechnik, Assigned: aleca)

References

Details

Attachments

(2 files)

Attached image attachment-pane.png (deleted) —

Steps to reproduce:

  • Have an account where signing and attachment of the key is enabled by default
  • Compose a new message
  • Disable signing of the message (which also disables the key being attached)

Actual result:
The attachment pane is still shown, with a label of "1 Attachment (0 bytes)" and no content.

Expected result:
The attachment pane disappears.

This is with Thunderbird 91.0b6

Blocks: tb91found
Flags: needinfo?(alessandro)

But we attach the key only at send time, so I don't understand why there was an attachment in the compose window to begin with?

Maybe crucial missing step then: sending fails because the external openPGP can't use the key it's asked to (smartcard not available). So I then disable signing, which leads to this.

So probably on a failed sending we remove the attached pgp key, and that removal doesn't properly trigger the update of the bucket UI.
I'm not too familiar with the PGP code handling the attached key, if you can point me to those methods I can see if I can take of fixing this.

Flags: needinfo?(alessandro)
Assignee: nobody → alessandro
Status: NEW → ASSIGNED
Target Milestone: --- → 93 Branch

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/3e92677b485f
Ensure the attachment pane is updated when GPG keys are added or removed. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

Comment on attachment 9236234 [details]
Bug 1725447 - Ensure the attachment pane is updated when GPG keys are added or removed. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #): bug 1695521
User impact if declined: When a PGP encrypted email fails during sending, the attachment pane remains visible with a wrong count number, as if the attached PGP key was still there.
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low

Attachment #9236234 - Flags: approval-comm-beta?

Comment on attachment 9236234 [details]
Bug 1725447 - Ensure the attachment pane is updated when GPG keys are added or removed. r=mkmelin

[Triage Comment]
Approved for beta

Attachment #9236234 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9236234 [details]
Bug 1725447 - Ensure the attachment pane is updated when GPG keys are added or removed. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #): bug 1695521
User impact if declined: When a PGP encrypted email fails during sending, the attachment pane remains visible with a wrong count number, as if the attached PGP key was still there.
Testing completed (on c-c, etc.): on c-c and beta
Risk to taking this patch (and alternatives if risky): low

Attachment #9236234 - Flags: approval-comm-esr91?

Comment on attachment 9236234 [details]
Bug 1725447 - Ensure the attachment pane is updated when GPG keys are added or removed. r=mkmelin

[Triage Comment]
Approved for esr91

Attachment #9236234 - Flags: approval-comm-esr91? → approval-comm-esr91+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: