Closed Bug 1463782 Opened 6 years ago Closed 6 years ago

Remote Settings throws out entries for toolkit@mozilla.org

Categories

(Toolkit :: Blocklist Policy Requests, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

Details

Attachments

(1 file)

Client-side tests assume that plugin/add-on items can be blocked for specific toolkit (ie gecko) versions. It seems this is not possible with the code in blocklist-clients.js as written, because it doesn't recognize the `toolkit@mozilla.org` ID, notices it doesn't match the app ID, and throws it away, without checking the toolkit version. We should change this.
I also removed the checks on minVersion because they too shouldn't be happening, and I don't think are currently happening on the server (per this comment https://github.com/mozilla-services/amo2kinto/pull/74/files#diff-7e59bae2f68a1c439e6c151596508cdeR72 ). I should probably have picked this up as part of the review in bug 1458920... :-(
Blocks: 1458920
Comment on attachment 8980011 [details] Bug 1463782 - allow toolkit apps in kinto blocklist, https://reviewboard.mozilla.org/r/246190/#review252336 ::: services/common/tests/unit/test_blocklist_clients.js:553 (Diff revision 1) > "guid": "xpcshell@tests.mozilla.org", > - "minVersion": "99999" > + "maxVersion": "20" > }], > }], > "id": "86771771-e803-4006-95e9-c9275d58b3d1" > }]}) Here it could be worth changing one of the record that matches on `guid: xpcshell@tests.mozilla.org` to `guid: toolkit@mozilla.org`, just to go through that new condition.
Attachment #8980011 - Flags: review?(mathieu) → review+
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/8a9d325e3285 allow toolkit apps in kinto blocklist, r=leplatrem
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: