Closed Bug 1580274 Opened 6 months ago Closed 5 months ago

Remove "ip" from proxy.json


(WebExtensions :: Request Handling, task)

Not set


(firefox71 fixed)

Tracking Status
firefox71 --- fixed


(Reporter: robwu, Assigned: mixedpuppy)



(Keywords: dev-doc-complete)


(1 file)

When proxy.onRequest was implemented (bug 1409878), its details were mostly copied from the webRequest API.

One of the fields in schemas/proxy.json is "ip", which is supposedly "The server IP address that the request was actually sent to."

This field is not set in the implementation, and it couldn't be because the IP cannot be known before a connection is made to a server, and the proxy.onRequest event is fired before that.

So "ip" should be removed from

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Component: General → Request Handling
Assignee: nobody → mixedpuppy

verify "ip" is not a part of details for proxy.onRequest in MDN docs.

Keywords: dev-doc-needed
Pushed by
remove unused ip value from proxy schema r=robwu
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71


Will this fix require manual validation? If yes, please provide some steps to reproduce in order to correctly test it and also, please set the "qe-verify+" flag. Otherwise, could the "qe-verify-" flag be added? Thanks!

Flags: needinfo?(mixedpuppy)
Flags: needinfo?(mixedpuppy) → qe-verify-

“ip” is removed from proxy.RequestDetails. Could you review and confirm that this fulfills the documentation needed request? However, I noticed that the proxy.json definition included "urlClassification", does this need to be added to proxy.RequestDetails?

Flags: needinfo?(mixedpuppy)

urlClassification is privileged, bug 1589494 is for making it public and it would be documented on that happening.

Flags: needinfo?(mixedpuppy)
You need to log in before you can comment on or make changes to this bug.