Closed
Bug 1625793
Opened 6 years ago
Closed 6 years ago
messages.query by folder shouldn't be allowed without accountsRead permission
Categories
(Thunderbird :: Add-Ons: Extensions API, defect)
Thunderbird
Add-Ons: Extensions API
Tracking
(thunderbird_esr68 fixed)
RESOLVED
FIXED
Thunderbird 76.0
| Tracking | Status | |
|---|---|---|
| thunderbird_esr68 | --- | fixed |
People
(Reporter: darktrojan, Assigned: darktrojan)
Details
Attachments
(1 file)
|
4.73 KB,
patch
|
mkmelin
:
review+
wsmwk
:
approval-comm-esr68+
|
Details | Diff | Splinter Review |
The extension shouldn't be able to get a folder object to pass to messages.query without the permission anyway, but it still should be blocked.
| Assignee | ||
Comment 1•6 years ago
|
||
Not in a rush to send this to 68, but it probably should go there.
Attachment #9137049 -
Flags: review?(mkmelin+mozilla)
Attachment #9137049 -
Flags: approval-comm-esr68?
Comment 2•6 years ago
|
||
Comment on attachment 9137049 [details] [diff] [review]
1625793-require-accountsread-1.diff
Review of attachment 9137049 [details] [diff] [review]:
-----------------------------------------------------------------
r=mkmelin
Attachment #9137049 -
Flags: review?(mkmelin+mozilla) → review+
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/f91579666b7a
Require accountsRead permission for functions involving folders. r=mkmelin
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 76.0
Comment 4•6 years ago
|
||
Comment on attachment 9137049 [details] [diff] [review]
1625793-require-accountsread-1.diff
Approved for ESR
Attachment #9137049 -
Flags: approval-comm-esr68? → approval-comm-esr68+
Comment 5•6 years ago
|
||
Updated•6 years ago
|
status-thunderbird_esr68:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•