Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: http-party/node-http-proxy Loading
base: 1.15.1
Choose a base ref
...
head repository: http-party/node-http-proxy Loading
compare: 1.15.2
Choose a head ref
  • 10 commits
  • 7 files changed
  • 6 contributors

Commits on Oct 4, 2016

  1. Add proxy-timeout option to documentation (#1075)

    http-proxy provides a [proxyTimeout option](https://github.com/nodejitsu/node-http-proxy/blob/master/lib/http-proxy/passes/web-incoming.js#L122) that allows us to set a timeout on the outgoing socket connection to the target. This timeout is very effective when the upstream target does not respond within an expected time.
    
    I had wasted a few hours searching for this option. Documenting this option can save others a significant amount of time.
    ashishdahiya authored and jcrugzz committed Oct 4, 2016
    Configuration menu
    Copy the full SHA
    d0a1588 View commit details
    Browse the repository at this point in the history

Commits on Oct 6, 2016

  1. not setting connection header in case of http2 as it is deprecated

    Niranjan Ojha authored and jcrugzz committed Oct 6, 2016
    Configuration menu
    Copy the full SHA
    2d01edc View commit details
    Browse the repository at this point in the history
  2. test case added

    Niranjan Ojha authored and jcrugzz committed Oct 6, 2016
    Configuration menu
    Copy the full SHA
    f5217d6 View commit details
    Browse the repository at this point in the history

Commits on Oct 22, 2016

  1. Fix browserification

    Browserify fails to resolve the "./http-proxy/" as "./http-proxy/index.js" but as "./http-proxy.js" (so nothing works)
    Beeing explicit here does not cost much for http-proxy, yet it's intrinsically complicated for browserify to fix (as trailing slash might be used as a pollyfill shim for native/non-natives addons i.e.  require('url/') vs require('url') )
    131 authored and jcrugzz committed Oct 22, 2016
    Configuration menu
    Copy the full SHA
    8eddf45 View commit details
    Browse the repository at this point in the history
  2. With a comment

    131 authored and jcrugzz committed Oct 22, 2016
    Configuration menu
    Copy the full SHA
    fbc2668 View commit details
    Browse the repository at this point in the history
  3. Do not rely on func.name (no scope)

    131 authored and jcrugzz committed Oct 22, 2016
    Configuration menu
    Copy the full SHA
    61c2889 View commit details
    Browse the repository at this point in the history
  4. Do not rely on func.name (no scope)

    131 authored and jcrugzz committed Oct 22, 2016
    Configuration menu
    Copy the full SHA
    d48f67e View commit details
    Browse the repository at this point in the history
  5. Expose full callback names

    131 authored and jcrugzz committed Oct 22, 2016
    Configuration menu
    Copy the full SHA
    220f5fb View commit details
    Browse the repository at this point in the history
  6. [fix] style nits

    jcrugzz committed Oct 22, 2016
    Configuration menu
    Copy the full SHA
    d0f1dfe View commit details
    Browse the repository at this point in the history
  7. 1.15.2

    jcrugzz committed Oct 22, 2016
    Configuration menu
    Copy the full SHA
    d822388 View commit details
    Browse the repository at this point in the history
Loading