Closed Bug 1662881 Opened 4 years ago Closed 4 years ago

Some OpenPGP signature states should report "unknown" rather than "mismatch"

Categories

(MailNews Core :: Security: OpenPGP, defect)

defect

Tracking

(thunderbird_esr78+ fixed, thunderbird81 fixed)

RESOLVED FIXED
82 Branch
Tracking Status
thunderbird_esr78 + fixed
thunderbird81 --- fixed

People

(Reporter: KaiE, Assigned: KaiE)

References

Details

Attachments

(1 file)

As reported in bug 1624676, if a correspondent's key has state "undecided", we currently set the state "mismatch" in the UI. This is wrong, we should set the state "unknown".

The same can be said if we don't have any meta information in the key/certificate, and therefore cannot make a comparison, this should also be "unknown".

State "mismatch" should be reserved to a scenario when the meta information (email) in the key/certificate clearly doesn't match the email sender information.

See Also: → 1624676
Summary: Some OpenPGP signature should report "unknown" rather than "mismatch" → Some OpenPGP signature states should report "unknown" rather than "mismatch"

Pushed by kaie@kuix.de:
https://hg.mozilla.org/comm-central/rev/52635fa7f970
Some OpenPGP signature states should report "unknown" rather than "mismatch". r=aleca

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch

Comment on attachment 9173757 [details]
Bug 1662881 - Some OpenPGP signature states should report "unknown" rather than "mismatch". r=aleca

[Triage Comment]
Approved for beta

Attachment #9173757 - Flags: approval-comm-beta+

Thanks for approving.

In the approval request, I wanted to say:

This change is a no-op unless icons are changed (currently both these states map to the same icon).

Comment on attachment 9173757 [details]
Bug 1662881 - Some OpenPGP signature states should report "unknown" rather than "mismatch". r=aleca

see previous comment

Attachment #9173757 - Flags: approval-comm-esr78?

Comment on attachment 9173757 [details]
Bug 1662881 - Some OpenPGP signature states should report "unknown" rather than "mismatch". r=aleca

[Triage Comment]
Approved for esr78.

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

Attachment

General

Created:
Updated:
Size: