Fixed crash with detect-no-csrf-before-method-override rule

[mathieumg]

Fixed crash with detect-no-csrf-before-method-override rule.

It's possible to do

const app = express();

[jlamendo]

Thanks @mathieumg - Merged.

[mathieumg]

Thanks! 😄 Are there any plans to publish in the near future? Last publication was 6 months ago. 😛

[priteshgupta]

Yes, this should be be published. On Node v6.2.2 I too am getting this error:

/Blah/node_modules/eslint-plugin-security/rules/detect-no-csrf-before-method-override.js:23
                if (node.callee.property.name === "methodOverride" && csrf) {
                                        ^

TypeError: Cannot read property 'name' of undefined
    at EventEmitter.CallExpression (/Blah/node_modules/eslint-plugin-security/rules/detect-no-csrf-before-method-override.js:23:41)
    at emitOne (events.js:101:20)
    at EventEmitter.emit (events.js:188:7)
    at NodeEventGenerator.enterNode (/Blah/node_modules/eslint/lib/util/node-event-generator.js:40:22)
    at CodePathAnalyzer.enterNode (/Blah/node_modules/eslint/lib/code-path-analysis/code-path-analyzer.js:607:23)
    at CommentEventGenerator.enterNode (/Blah/node_modules/eslint/lib/util/comment-event-generator.js:97:23)
    at Controller.traverser.traverse.enter (/Blah/node_modules/eslint/lib/eslint.js:902:36)
    at Controller.__execute (/Blah/node_modules/estraverse/estraverse.js:397:31)
    at Controller.traverse (/Blah/node_modules/estraverse/estraverse.js:501:28)
    at Controller.Traverser.controller.traverse (/Blah/node_modules/eslint/lib/util/traverser.js:36:33)

[bonesoul]

this is also crashing eslint for me.