fix(express): use app.set for "trust proxy" (#10503)

noam-honig · ndom91 · balazsorban44 · commit 783599ba2255 · 2024-04-08T17:55:23.000+02:00
Co-authored-by: Nico Domino &lt;yo@ndo.dev&gt;
diff --git a/packages/frameworks-express/src/index.ts b/packages/frameworks-express/src/index.ts
@@ -22,7 +22,8 @@
  * const app = express()
  *
  * // If app is served through a proxy, trust the proxy to allow HTTPS protocol to be detected
- * app.use('trust proxy')
+ * // https://expressjs.com/en/guide/behind-proxies.html
+ * app.set('trust proxy', true) 
  * app.use("/auth/*", ExpressAuth({ providers: [ GitHub ] }))
  * ```
  *

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,8 @@`
`22`	`22`	`* const app = express()`
`23`	`23`	`*`
`24`	`24`	`* // If app is served through a proxy, trust the proxy to allow HTTPS protocol to be detected`
`25`		`- * app.use('trust proxy')`
	`25`	`+ * // https://expressjs.com/en/guide/behind-proxies.html`
	`26`	`+ * app.set('trust proxy', true)`
`26`	`27`	`* app.use("/auth/*", ExpressAuth({ providers: [ GitHub ] }))`
`27`	`28`	* ```
`28`	`29`	`*`