Closed Bug 1218282 Opened 4 years ago Closed 4 years ago

Add-ons: copy content_scripts field from manifest.json to update.webapp (mini-manifest)

Categories

(Marketplace Graveyard :: API, defect, P2)

Avenir
defect

Tracking

(Not tracked)

RESOLVED FIXED
2015-11-03

People

(Reporter: mikehenrty, Assigned: mat)

References

Details

(Whiteboard: [qa-])

In order to make bug 1216446 work (displaying warning on installation for add-ons that inject into the system app), the system needs to know the value of the content_scripts field in the add-ons manifest.json. Right now the only way to do that is to download the package, unzip it, and examine it's contents. But this defeats the purpose of the installation confirm dialog. So to make this bug work, we would need the Marketplace team to copy the content_scripts field from manifest.json to the update.webapp.
Maris, can you help me figure out if this is doable from Marketplace side for the 2.5 timeframe?

Note, that we can put the code into the system app to read the content_scripts field of the update.manifest, and then at any time (even beyond Nov. 2) Marketplace can add that field to the add-on manifests. But it would be nice to have something to test against before launch.
Flags: needinfo?(mars)
I can add it easily right now if it's just copying it out as we store a copy of the original manifest in db and generate the update manifest on the fly.
Flags: needinfo?(mars)
Assignee: nobody → mpillard
Priority: -- → P2
Target Milestone: --- → 2015-11-03
https://github.com/mozilla/zamboni/pull/3410
Status: NEW → ASSIGNED
Thanks Mat!
Fixed in https://github.com/mozilla/zamboni/commit/d360650b84b926b7e50ae22bac93fdda992583f2

It will land on -dev in a few minutes, later on stage/prod.
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.