Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

proxy json requests with node express

I use the following node-express code to proxy requests from a web server to an API server:

app.use('/api', function(req, res) {
  var url = 'http://my.domain.com/api' + req.url;
  req.pipe(request(url)).pipe(res);
});

This works well for simple requests of any verb (get, post, etc...), however once I send 'Content-type': 'application/json' requests, it hangs on the pipe line.

Why does this simple node-express proxy code hang on json requests?
How can it be altered to support them?

like image 552
Jonathan Livni Avatar asked Sep 30 '14 13:09

Jonathan Livni


2 Answers

You need to move that custom middleware function before this line so that it's executed before any of the body parsers. This makes sure that the request data is still there for piping to request(url) in your custom middleware.

The cause of the hanging currently is that req has no data to write to request(url) (because the body parsing middleware already read all of the request data and parsed it) and so it never calls .end() on the request(url) stream. This means that the request to url never completes because it's just sitting there waiting for data that it will never get.

like image 101
mscdex Avatar answered Nov 04 '22 14:11

mscdex


In the case of a post request, the following construct works:

app.post('/api/method', (req, res) => {
  req.pipe(request.post(someUrl, { json: true, body: req.body }), { end: false }).pipe(res);
}

This is of course relevant if you're using the bodyparser middleware.

like image 4
SimoAmi Avatar answered Nov 04 '22 14:11

SimoAmi