node-express error : express deprecated res.send(status): Use res.sendStatus(status) instead

Question

I am trying to send an integer via response.send() but I keep getting this error

express deprecated res.send(status): Use res.sendStatus(status) instead

I am not sending a Status, my code is

app.get('/runSyncTest' , function(request, response){  

var nodes = request.query.nodes;
var edges = request.query.edges;
if (edges == "" ){
    edges = []
}

userStory.userStory(nodes,edges);
connection.query('SELECT MAX(id) as id FROM report ', function(err,results, fields) {
                idTest = results[0].id
                response.send (idTest)
});

});

just to avoid warning, how about returning JSON like `response.send ({max:idTest})` — Haris Hasan, May 30 '15 at 18:57
Regarding `res.send()`, [the fine manual](http://expressjs.com/4x/api.html#res.send) states: _"The body parameter can be a Buffer object, a String, an object, or an Array."_. So integers aren't directly supported and need to be converted to one of those types first. — robertklep, May 30 '15 at 19:03

score 92 · Accepted Answer · answered Sep 16 '15 at 07:22

92

You could try this:

res.status(200).send((results[0].id).toString());

Guys are right - it doesn't allow numbers. Prooflink: http://expressjs.com/4x/api.html#res.send

answered Sep 16 '15 at 07:22

Arif Dewi

2,222
26
14

2

thanks it help ,my problem is trying to send extra content which are not string – Ashad Nasim Mar 10 '19 at 13:51
@letie but I also do not remember what I was asking at that time, 2 years back... – Ashad Nasim Sep 08 '22 at 06:12

score 25 · Answer 2 · answered Jun 24 '16 at 06:45

25

This is because you are sending numeric value in the res.send.

You could send a json object or convert it to string.

answered Jun 24 '16 at 06:45

Jerome Miranda

321
4
8

score 7 · Answer 3 · answered Sep 28 '15 at 08:34

(as mentioned in the comments already)

The manual states:

The body parameter can be a Buffer object, a String, an object, or an Array.

So integers aren't directly supported and need to be converted to one of those types first. For instance:

response.send(String(idTest));

score 5 · Answer 4 · answered Mar 02 '17 at 16:21

5

As long as you're not sending String or Object/Array data you get an error. Solution convert your data to string:

app.get('/runSyncTest', function(req, res) {
    var number = 5000;
    res.send((number).toString()); //Number is converted with toString()
});

answered Mar 02 '17 at 16:21

RegarBoy

3,228
1
23
44

This answer works without any error or warning. Clearly, this is a bug in the error handling of Express. it has a type error but showed the wrong error (deprecated...) – Milad Abooali Jun 12 '23 at 06:40
@MiladAbooali if you are using JS, there're no type checks by default in JS, thus TS was introduced. – RegarBoy Jun 12 '23 at 09:08
2

,Thanks for reminding that. but I'm not talking about type checks. I meant from a bug is the wrong error message. This is about the Express send method, not JS basics. We expect to see anything like empty input or wrong input but not a deprecated notice! This msg is not related to the error at all – Milad Abooali Jun 13 '23 at 01:09

Dere Sagar · Answer 5 · 2021-12-14T12:30:52.607

5

Its Deprecated, Express 5 No Longer Supports The Signature Like -

res.json(200, {
   result: result
});

Instead, use a below method, Means you only need to change the format of sending responses.

res.status(statusCode).json(result);

Example -

res.status(200).json({'success' : true, 'result': result})

edited Dec 14 '21 at 12:30

answered May 13 '19 at 09:23

Dere Sagar

1,719
1
10
7

score 3 · Answer 6 · edited Jul 09 '18 at 22:41

3

Use like this,

res.status(404).send('Page Not found');

edited Jul 09 '18 at 22:41

ddmps

4,350
1
19
34

answered Apr 14 '16 at 09:24

Anshul Bisht

1,644
20
21

3

this doesn't solve the issue at all; you are literally just sending an error in response – dlq Dec 31 '20 at 05:21

score 1 · Answer 7 · answered Oct 19 '20 at 13:46

I am getting a deprecated warning in node-express typescript project due to res.send(403);

Solution 1 -

res.sendStatus(403);

Solution 2 -

res.status(403).json(
        {
            'success': false,
            'result': 'forbidden'
        }
    )

score 0 · Answer 8 · answered Mar 11 '23 at 20:59

if you have in your code:

app.use(bodyParser.json());

then you should always return a 'JSON' with the response, for example:

the wrong answer for get request is:

router.get("/", async (req, res) => {
    const result = 3;
    res.send(result);
});

the right answer should be:

router.get("/", async (req, res) => {
    const result = 3;
    res.send({ result });
});

score -2 · Answer 9 · answered Aug 10 '22 at 22:19

-2

Simply converting it to a string works great.

response.send(idTest.toString()

answered Aug 10 '22 at 22:19

Zach Atwood

1

1

This is already covered in previous answers – klutt Aug 15 '22 at 07:32
And it's also missing a ')' – letie Sep 07 '22 at 16:28

node-express error : express deprecated res.send(status): Use res.sendStatus(status) instead

9 Answers9

Linked

Related