Skip to content

error logging ignores router target#242

Closed
bender316 wants to merge 1 commit intochimurai:masterfrom
bender316:master
Closed

error logging ignores router target#242
bender316 wants to merge 1 commit intochimurai:masterfrom
bender316:master

Conversation

@bender316
Copy link
Copy Markdown

Hi,

I'm using a router function to override the target. If an error occurs the old target is written to the log which can be really misleading.
I changed the logging so that it first gets the router target and then uses one of the available targets for output.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.009%) to 97.076% when pulling 93c8aaf on bender316:master into 66396d4 on chimurai:master.

2 similar comments
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.009%) to 97.076% when pulling 93c8aaf on bender316:master into 66396d4 on chimurai:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 13, 2018

Coverage Status

Coverage increased (+0.009%) to 97.076% when pulling 93c8aaf on bender316:master into 66396d4 on chimurai:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 13, 2018

Coverage Status

Coverage increased (+0.01%) to 93.384% when pulling 7a16955 on bender316:master into 3b97308 on chimurai:master.

@lerit
Copy link
Copy Markdown

lerit commented Aug 22, 2019

@bender316
would you please fix this conflict ?

@bender316
Copy link
Copy Markdown
Author

@lerit / @chimurai
I've updated (rebased) the PR. Please check it again.

@textbook
Copy link
Copy Markdown

This just came up in a Stack Overflow question, shame it didn't get merged 😕

@bender316
Copy link
Copy Markdown
Author

I'm not working with http-proxy-middleware anymore. Maybe I find some time to rebase it but this really could take some time.

Maybe it gets merged sometime.

@textbook
Copy link
Copy Markdown

textbook commented Nov 3, 2020

That's fair enough @bender316 - I'd be happy to create a new PR for this if a maintainer (@lerit?) thinks it would get merged.

@chimurai
Copy link
Copy Markdown
Owner

fixed in v1.1.2

@chimurai chimurai closed this Apr 14, 2021
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.

5 participants