Skip to content
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

Modified the ad-hoc proxy lookup to use _getKey(), rather than the error-prone in-line method. #164

Merged
merged 1 commit into from
Dec 14, 2011
Merged

Modified the ad-hoc proxy lookup to use _getKey(), rather than the error-prone in-line method. #164

merged 1 commit into from
Dec 14, 2011

Conversation

elfsternberg
Copy link
Contributor

Modified the ad-hoc proxy lookup to use _getKey(), rather than the error-prone in-line method.

_getKey() will look for options.target as well as options.host:options.port, and so is useful for a segmented proxy server where the destination proxies are not constructed before first references.

error-prone in-line method.

_getKey() will look for options.target as well as
options.host:options.port, and so is useful for a segmented
proxy server where the destination proxies are not constructed before
first references.
@AvianFlu
Copy link
Contributor

Thanks! We'll get this reviewed as soon as possible.

indexzero added a commit that referenced this pull request Dec 14, 2011
…7aa50452a9d40

Modified the ad-hoc proxy lookup to use _getKey(), rather than the error-prone in-line method.
@indexzero indexzero merged commit d1b19a1 into http-party:master Dec 14, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants