Replace any by mixed in express_v4.x.x.js (#1737)

Using `any` will effectively disable the type checker anywhere you use the `req.body` property, which is dangerous. By declaring it as a `mixed` type, it allows you to use it, but you'll have to make sure that any assumptions you make about its contents are checked first, which is the way to go.
flow-typed · Jan 29, 2018 · 9f7cf2a · 9f7cf2a
1 parent 03669c2
commit 9f7cf2a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/definitions/npm/express_v4.16.x/flow_v0.32.x-/express_v4.16.x.js b/definitions/npm/express_v4.16.x/flow_v0.32.x-/express_v4.16.x.js
@@ -18,7 +18,7 @@ declare type express$RequestParams = {
 
 declare class express$Request extends http$IncomingMessage mixins express$RequestResponseBase {
  baseUrl: string;
- body: any;
+ body: mixed;
  cookies: { [cookie: string]: string };
  connection: Socket;
  fresh: boolean;

diff --git a/definitions/npm/express_v4.x.x/flow_v0.32.x-/express_v4.x.x.js b/definitions/npm/express_v4.x.x/flow_v0.32.x-/express_v4.x.x.js
@@ -18,7 +18,7 @@ declare type express$RequestParams = {
 
 declare class express$Request extends http$IncomingMessage mixins express$RequestResponseBase {
  baseUrl: string;
- body: any;
+ body: mixed;
  cookies: {[cookie: string]: string};
  connection: Socket;
  fresh: boolean;