feat: add trust_proxy config option for reverse proxy deployments#414
feat: add trust_proxy config option for reverse proxy deployments#414ppastur wants to merge 1 commit into
Conversation
Resolves ryanlelek#413 When deployed behind a reverse proxy, express-rate-limit warns that trust proxy is not set. Add a trust_proxy config option that calls app.set('trust proxy', value) when present.
|
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #414 +/- ##
==========================================
+ Coverage 62.58% 72.18% +9.59%
==========================================
Files 33 33
Lines 850 852 +2
Branches 236 237 +1
==========================================
+ Hits 532 615 +83
+ Misses 269 207 -62
+ Partials 49 30 -19 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi @ryanlelek , That said, if there's anything you'd like me to change or fix on my end, happy to do so — just let me know! |



Resolves #413
When deployed behind a reverse proxy, express-rate-limit warns that trust proxy is not set. Add a trust_proxy config option that calls app.set('trust proxy', value) when present.