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.

Sign up for GitHub

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

Jump to bottom

Add support for PROXY protocol #400

Closed
niftylettuce opened this issue Jul 6, 2020 · 2 comments
Closed

Add support for PROXY protocol #400

niftylettuce opened this issue Jul 6, 2020 · 2 comments

Comments

@niftylettuce
Copy link
Collaborator

This needs implemented across @ladjs/web, @ladjs/proxy, and @ladjs/api.

https://github.com/cusspvz/proxywrap

It should be opt-in only.
@niftylettuce
Copy link
Collaborator Author

Without this, then PROXY protocol will not work in the future behind load balancers.

niftylettuce added a commit to ladjs/proxy that referenced this issue Jul 6, 2020
@niftylettuce
feat: added PROXY protocol support per <ladjs/lad#400>
d2a4102
niftylettuce added a commit to ladjs/shared-config that referenced this issue Jul 6, 2020
@niftylettuce
feat: added PROXY protocol support per <ladjs/lad#400>
3c92317
@niftylettuce
Copy link
Collaborator Author

WEB_PROXY_PROTOCOL, PROXY_PROTOCOL, and API_PROXY_PROTOCOL are now env vars per @ladjs/web @ladjs/api (via @ladjs/shared-config) -- and also @ladjs/api

niftylettuce added a commit to ladjs/api that referenced this issue Jul 6, 2020
@niftylettuce
feat: added PROXY protocol support per <ladjs/lad#400>
a65078e
niftylettuce added a commit to ladjs/web that referenced this issue Jul 6, 2020
@niftylettuce
feat: added PROXY protocol support per <ladjs/lad#400>
9738114
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@niftylettuce @shaunwarman