OCram85/code-server
Archived
1
0
Fork 0
Code Pull Requests 2 Activity

routes/index.ts: register proxy routes before body-parser

Browse Source 
Any json or urlencoded request bodies were being consumed by body-parser
before they could be proxied. That's why requests without Content-Type
were proxied correctly as body-parser would not consume their body.

This allows the http-proxy package to passthrough the request body correctly
in all instances.

Closes #2377
This commit is contained in:
Anmol Sethi 2021-01-20 17:32:47 -05:00
parent f5cf3fd331
commit 58d72d53a1
No known key found for this signature in database
GPG Key ID: 8CEF1878FF10ADEB
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/node/routes/index.ts
View File

@ -65,9 +65,6 @@ export const register = async (
app.use(cookieParser()) app.use(cookieParser())
wsApp.use(cookieParser()) wsApp.use(cookieParser())
app.use(bodyParser.json())
app.use(bodyParser.urlencoded({ extended: true }))
const common: express.RequestHandler = (req, _, next) => { const common: express.RequestHandler = (req, _, next) => {
// /healthz|/healthz/ needs to be excluded otherwise health checks will make // /healthz|/healthz/ needs to be excluded otherwise health checks will make
// it look like code-server is always in use. // it look like code-server is always in use.
@ -106,6 +103,12 @@ export const register = async (
app.use("/", domainProxy.router) app.use("/", domainProxy.router)
wsApp.use("/", domainProxy.wsRouter.router) wsApp.use("/", domainProxy.wsRouter.router)
app.use("/proxy", proxy.router)
wsApp.use("/proxy", proxy.wsRouter.router)
app.use(bodyParser.json())
app.use(bodyParser.urlencoded({ extended: true }))
app.use("/", vscode.router) app.use("/", vscode.router)
wsApp.use("/", vscode.wsRouter.router) wsApp.use("/", vscode.wsRouter.router)
app.use("/vscode", vscode.router) app.use("/vscode", vscode.router)
@ -121,9 +124,6 @@ export const register = async (
}) })
} }
app.use("/proxy", proxy.router)
wsApp.use("/proxy", proxy.wsRouter.router)
app.use("/static", _static.router) app.use("/static", _static.router)
app.use("/update", update.router) app.use("/update", update.router)