如何修复 codacy 警报“通用对象注入接收器”

Question

下面是我的代码。我不认为有任何问题。

我怎样才能愚弄 codacy？如果我不能使用obj[key]，那么这到底是什么东西？我无法避免[]。

handleClick = (e, titleProps) => {
     const { index } = titleProps
     const newVal = this.state.activeIndexObj[index]? false: true
     let activeIndexObj = {...this.state.activeIndexObj}
     activeIndexObj[index] = newVal
     // Generic Object Injection Sink (security/detect-object-injection)

Answer 1

你只需要将索引解析成整数

activeIndexObj[parseInt(index)] = newVal

黑客可能会注入函数或原型链，这就是出现此安全错误的原因。

Answer 2

@luca (Why is it bad pratice calling an array index with a variable?) 在评论中链接的问题解释了使用变量访问数组索引的问题。这是一个安全问题。

如果您允许将未经验证的输入用作数组索引，您的应用程序可能会崩溃。即使您验证了索引，重构代码并跳过验证也只是时间问题。因此建议避免使用此类代码。一种推荐的解决方案是使用地图：https://stackoverflow.com/a/44882765/4398050

如果你不想知道这个问题，可以忽略代码用户界面中的问题：https://support.codacy.com/hc/en-us/articles/207279979-Issues#2-remove-pattern