Skip to content

Pass options to proxyRes event.#1283

Open
j wants to merge 1 commit intohttp-party:masterfrom
j:options-to-proxyRes
Open

Pass options to proxyRes event.#1283
j wants to merge 1 commit intohttp-party:masterfrom
j:options-to-proxyRes

Conversation

@j
Copy link
Copy Markdown

@j j commented Jul 20, 2018

I wanted to be able to use the built-in web-outgoing passes for including default headers into my final response. Only thing is that options weren't available in it. This fixes it.

I also moved the web outgoing passes function into common for code-reuse.

@j
Copy link
Copy Markdown
Author

j commented Jul 20, 2018

/ping @jcrugzz

@j
Copy link
Copy Markdown
Author

j commented Jul 23, 2018

/ping @indexzero

@codecov-io
Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@a3fe02d). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##             master   #1283   +/-   ##
========================================
  Coverage          ?   92.4%           
========================================
  Files             ?       6           
  Lines             ?     316           
  Branches          ?       0           
========================================
  Hits              ?     292           
  Misses            ?      24           
  Partials          ?       0
Impacted Files Coverage Δ
lib/http-proxy/passes/web-incoming.js 98.21% <100%> (ø)
lib/http-proxy/common.js 97.29% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a3fe02d...8adbad8. Read the comment docs.

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