-
Notifications
You must be signed in to change notification settings - Fork 22.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
proxy.ProxyInfo.proxyAuthorizationHeader HTTP support #32536
proxy.ProxyInfo.proxyAuthorizationHeader HTTP support #32536
Conversation
Preview URLs
External URLs (1)URL:
(comment last updated: 2024-04-04 17:54:08) |
I think a note to state that the support was added in Firefox 125 would avoid unnecessary confusion for the developers (especially since it is not out as standard release yet). |
@rebloor @Rob--W could that fit into a note added into a browser-compat-data/webextensions/api/proxy.json? wdyt? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks @rebloor
@rpl added BCD note in mdn/browser-compat-data#22668. I'll also add release note shortly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Amended the description of 'proxy.ProxyInfo.proxyAuthorizationHeader' to note that it also supports setting for HTTP proxies. This change was made in Bug 1794464
Allow HTTP authentication in proxy.onRequest.
Related issues and pull requests
Fixes #32475