Closed
Bug 1130795
Opened 9 years ago
Closed 1 month ago
Add latest jpm-core bootstrap.js hash to list of allowed sdk bootstraps
Categories
(addons.mozilla.org Graveyard :: Add-on Validation, defect)
addons.mozilla.org Graveyard
Add-on Validation
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: evold, Unassigned)
References
Details
In order to have add-ons made with `jpm` to be accepted as "jetpack" or an sdk based add-on we need to have the latest jpm-core bootstrap.js file's hash added to the list of those accepted as sdk bootstrap files. This new bootstrap file is here https://github.com/mozilla/jpm-core/blob/master/data/bootstrap.js
Reporter | ||
Updated•9 years ago
|
Reporter | ||
Updated•9 years ago
|
Severity: normal → blocker
Updated•9 years ago
|
Assignee: nobody → kmaglione+bmo
Reporter | ||
Comment 1•9 years ago
|
||
Hey Kris, Is there any way we can force push this new hash in to the validator? I'd like to blog about jpm soon and this is the only blocker. Also, I'm not sure why bug 1069570 blocks this.
Flags: needinfo?(kmaglione+bmo)
Comment 2•9 years ago
|
||
So, what we need to do is insert the sha256 sum (?) of https://github.com/mozilla/jpm-core/blob/master/data/bootstrap.js to https://github.com/mozilla/amo-validator/blob/master/validator/testcases/jetpack_data.txt right?
Updated•9 years ago
|
Flags: needinfo?(evold)
Comment 3•9 years ago
|
||
No. That file is going away. I don't want to do any more updates to the old hash metadata before the replacement code lands.
Flags: needinfo?(kmaglione+bmo)
Flags: needinfo?(evold)
Reporter | ||
Comment 4•9 years ago
|
||
(In reply to Kris Maglione [:kmag] from comment #3) > No. That file is going away. I don't want to do any more updates to the old > hash metadata before the replacement code lands. what bug has this replacement code?
Flags: needinfo?(kmaglione+bmo)
Reporter | ||
Comment 6•9 years ago
|
||
(In reply to Kris Maglione [:kmag] from comment #5) > See dependency list. Kris this doesn't answer my question from comment 1, I'm wondering where the patch we are waiting for is.
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(kmaglione+bmo)
Updated•9 years ago
|
Severity: blocker → normal
Flags: needinfo?(kmaglione+bmo)
Assignee | ||
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
Updated•8 years ago
|
Assignee: kmaglione+bmo → nobody
You need to log in
before you can comment on or make changes to this bug.
Description
•