Closed
Bug 1244805
Opened 9 years ago
Closed 9 years ago
Relative URLs cause access errors at install time
Categories
(WebExtensions :: Untriaged, defect)
WebExtensions
Untriaged
Tracking
(firefox47 fixed)
VERIFIED
FIXED
mozilla47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: kmag, Assigned: kmag)
References
Details
Attachments
(1 file)
We resolve relative URLs in manifests relative to the filesystem URL when validating them from the add-on manager. Since file: URLs aren't loadable by the null principal, this causes access errors.
This should have been caught by our tests, but they don't currently run the same startup checks as the add-on manager for every test add-on.
Assignee | ||
Updated•9 years ago
|
Severity: normal → blocker
Assignee | ||
Comment 1•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/33065/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/33065/
Attachment #8714398 -
Flags: review?(wmccloskey)
Comment on attachment 8714398 [details]
MozReview Request: Bug 1244805: [webext] Don't resolve relative URLs in manifests unless extension is running. r?billm
https://reviewboard.mozilla.org/r/33065/#review29829
Attachment #8714398 -
Flags: review?(wmccloskey) → review+
Assignee | ||
Comment 3•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/184854a1ca0e5a89a380d1976dd1f678d633e48f
Bug 1244805: [webext] Don't resolve relative URLs in manifests unless extension is running. r=billm
Comment 4•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Comment 6•9 years ago
|
||
Could you please provide some steps in order to manually verify this bug?
Flags: needinfo?(kmaglione+bmo)
Assignee | ||
Comment 7•9 years ago
|
||
This bug has been pretty thoroughly verified, I think.
Status: RESOLVED → VERIFIED
Flags: needinfo?(kmaglione+bmo)
Updated•7 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•