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
add optional parameter to getHttpProxyConfig allow bypass filter. #324
Conversation
node/task.ts
Outdated
}; | ||
let bypass: boolean = false; | ||
if (requestUrl) { | ||
this._httpProxyBypassHosts.forEach(bypassHost => { |
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.
Didn't see this in the code? How is this different than proxyBypassHosts?
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.
Does that include wildcards, ip ranges etc as well?
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.
it's a list of regex patterns
@@ -1563,19 +1563,33 @@ export interface ProxyConfiguration { | |||
* | |||
* @return ProxyConfiguration | |||
*/ | |||
export function getHttpProxyConfiguration(): ProxyConfiguration { | |||
export function getHttpProxyConfiguration(requestUrl?: string): ProxyConfiguration { |
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.
Should this be URL?
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.
i might just leave it as string since the bypass list is always an array of strings. :D
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.
OK, You could do requestUrl.host if it was a URL() though.
@TingluoHuang, |
Plan is to click merge which I just did. |
@bryanmacfarlane, |
:) |
tested.