Skip to content

Fixes xForward dependency.#219

Merged
stefanoborini merged 2 commits into
masterfrom
feature/fix-x-forward-proxy-dependency-0908-1131
Sep 8, 2016
Merged

Fixes xForward dependency.#219
stefanoborini merged 2 commits into
masterfrom
feature/fix-x-forward-proxy-dependency-0908-1131

Conversation

@stefanoborini

Copy link
Copy Markdown
Contributor

Current release of configurable-http-proxy has a bug that inverts the logic
of x-forward addition. This ruins the functionality of anything reverseproxied,
specifically the file transfer (which relies on these headers to produce the correct
urls)

Current release of configurable-http-proxy has a bug that inverts the logic
of x-forward addition. This ruins the functionality of anything reverseproxied,
specifically the file transfer (which relies on these headers to produce the correct
urls)
@codecov-io

codecov-io commented Sep 8, 2016

Copy link
Copy Markdown

Current coverage is 93.22% (diff: 100%)

Merging #219 into master will not change coverage

@@             master       #219   diff @@
==========================================
  Files            73         73          
  Lines          3175       3175          
  Methods           0          0          
  Messages          0          0          
  Branches        209        209          
==========================================
  Hits           2960       2960          
  Misses          151        151          
  Partials         64         64          

Powered by Codecov. Last update 2bfc61d...607201e

@stefanoborini stefanoborini merged commit c37cc3f into master Sep 8, 2016
@stefanoborini stefanoborini deleted the feature/fix-x-forward-proxy-dependency-0908-1131 branch September 8, 2016 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants