From 04561912706da7fc0446a26f7fc041febef8b55a Mon Sep 17 00:00:00 2001 From: Julian Simioni Date: Mon, 13 Apr 2026 14:28:41 -0400 Subject: [PATCH] feat(jsonp): Remove JSONP middleware JSONP is a relic of the ancient past. If you're using it, you shouldn't be. --- app.js | 1 - middleware/jsonp.js | 23 ----------------------- 2 files changed, 24 deletions(-) delete mode 100644 middleware/jsonp.js diff --git a/app.js b/app.js index f27d173c8..42ec1af66 100644 --- a/app.js +++ b/app.js @@ -12,7 +12,6 @@ app.use( require('./middleware/headers') ); app.use( require('./middleware/cors') ); app.use( require('./middleware/robots') ); app.use( require('./middleware/options') ); -app.use( require('./middleware/jsonp') ); /** ----------------------- routes ----------------------- **/ diff --git a/middleware/jsonp.js b/middleware/jsonp.js deleted file mode 100644 index 3f06dbf68..000000000 --- a/middleware/jsonp.js +++ /dev/null @@ -1,23 +0,0 @@ -function middleware(req, res, next){ - - // store old json function - var json = res.json.bind(res); - - // replace with jsonp aware function - res.json = function( data ){ - - // jsonp - if( req.query && req.query.callback ){ - res.header('Content-type','application/javascript'); - return res.send( req.query.callback + '('+ JSON.stringify( data ) + ');' ); - } - - // regular json - res.header('Content-type','application/json'); - return json( data ); - }; - - next(); -} - -module.exports = middleware; \ No newline at end of file