Open
Bug 1053364
Opened 10 years ago
Updated 9 years ago
Don't warn about nsIProtocolProxyService, nsIProxyAutoConfig, or newProxiedChannel for addons explicitly supporting current application versions
Categories
(addons.mozilla.org Graveyard :: Add-on Validation, defect, P4)
addons.mozilla.org Graveyard
Add-on Validation
Tracking
(Not tracked)
REOPENED
People
(Reporter: davemgarrett, Unassigned)
References
Details
+++ This bug was initially created as a clone of Bug #795168 +++
The addon validator flags usage of nsIProtocolProxyService, nsIProxyAutoConfig, or newProxiedChannel (bug 795168) due to the sync API being removed in favor of async only (bug 769764).
However, the detection isn't particularly smart. I'm still getting notices about this when I upload new versions, and an AMO editor actually mentioned it to me in an email. I think it's long past time to quash these.
What I propose is only flagging these API changes in the validator if the addon's install.rdf does not explicitly set a max supported application version that is current. So, if it has Firefox 31 (current ESR), or equivalent other application version, or later in its max version, then this warning would not apply any longer.
Alternatively, the warning could just be retired altogether, but I don't see a reason not to keep it in there for people updating particularly old addons.
Reporter | ||
Updated•10 years ago
|
Whiteboard: [fx18]
Comment 1•10 years ago
|
||
Thanks for filing this. Due to resource constraints we are closing bugs which we won't realistically be able to fix. If you have a patch that applies to this bug please reopen.
For more info see http://micropipes.com/blog/2014/09/24/the-great-add-on-bug-triage/
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Reporter | ||
Comment 2•10 years ago
|
||
Reopening. This is just routine validator updating. The warning just needs to be updated. At bare minimum, just remove it, as it's not needed anymore.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Comment 3•9 years ago
|
||
Thanks for the info. Removing these sounds like a good thing to do
Priority: -- → P4
Assignee | ||
Updated•9 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•