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

feat: add proxyHeadersIgnore in default options #175

Merged
merged 1 commit into from Nov 7, 2018

Conversation

Projects
None yet
3 participants
@lefuturiste
Copy link
Contributor

lefuturiste commented Nov 1, 2018

From a suggestion from @manniL on the nuxt community discord. I added cf-ray and cf-connecting-ip to default proxy ignored header.

This solution fixed my own problem with cloudflare proxy and this is also the solution for this issue (#20)

@manniL manniL requested a review from pi0 Nov 1, 2018

@manniL

manniL approved these changes Nov 1, 2018

@pi0

This comment has been minimized.

Copy link
Member

pi0 commented Nov 1, 2018

It should be better-supporting regex and wildcard ignores but i can merge/publish it if it is urgent.

@lefuturiste

This comment has been minimized.

Copy link
Contributor Author

lefuturiste commented Nov 1, 2018

why ?

@manniL manniL changed the title feat: add proxyHeadersIgnore in default options (#20) feat: add proxyHeadersIgnore in default options Nov 5, 2018

@pi0

This comment has been minimized.

Copy link
Member

pi0 commented Nov 7, 2018

@lefuturiste For supporting more patterns :) Merging as-is for now :) Thanks for contribution.

@pi0 pi0 merged commit 3ae5416 into nuxt-community:dev Nov 7, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.