kzilla.xyz icon indicating copy to clipboard operation
kzilla.xyz copied to clipboard

Published 20 hours ago • verified publisher icon srm-kzilla

Body parser has been deprecated

Open shawshankkumar opened this issue 2 years ago • 2 comments

Expected Behavior

Express has a built in parser, the code should use that.

Current Behavior

Bodyparser is being used.

shawshankkumar avatar Oct 22 '22 15:10 shawshankkumar

Hi, there. I would like to work on this issue. Can it be assigned to me?

gokullan avatar Oct 23 '22 08:10 gokullan

@gokullan Hello, yes you can work on this!

sm-sami avatar Oct 27 '22 13:10 sm-sami

Hi, there. I have made the required change in the server.ts file. However, I am unable to completely remove body-parser from node-modules, package.json, etc. Below are the problems on which I would like some support.

  • When I ran npm uninstall body-parser or npm uninstall @types/body-parser, it resulted in the deletion of a bunch of files which I don't think should've happened (Screenshot of git status after uninstalling body-parser is attached below)
  • I am unsure of how to set the environment variables (GAPI_PRIVATE_KEY, GAPI_VIEW_ID, ...) due to which I am unable to test the behaviour of my change. image

gokullan avatar Nov 03 '22 12:11 gokullan