- 필수 기능
- 앱 내
- 서비스 관리
- 인프라스트럭처
- 애플리케이션 성능
- 디지털 경험
- 소프트웨어 제공
- 보안
- 로그 관리
- 관리
- 인프라스트럭처
- ci
- containers
- csm
- ndm
- otel_guides
- overview
- slos
- synthetics
- tests
- 워크플로
ID: javascript-express/external-filename-upload
Language: JavaScript
Severity: Warning
Category: Security
CWE: 73
Using unsanitized user input in a sendFile
method can allow attackers to access unintended resources.
Set the root
option directly in your sendFile
options will make this rule not report a violation.
app.post("/upload", (req, res) => {
res.sendFile(req.params.filename)
// options passed, but no root set
res.sendFile(req.params.filename, { maxAge: 0 })
// options passed, but no root set, and a callback is set
res.sendFile(req.params.filename, { maxAge: 0 }, (err) => console.log(err))
})
app.post("/upload", (req, res) => {
res.sendFile("foo")
const options = { maxAge: 0, root: path.join(__dirname, "upload") }
// options with root set
res.sendFile(req.params.filename, options)
res.sendFile(req.params.filename, { maxAge: 0, root: path.join(__dirname, "upload") })
// options with root set (and a callback is set)
res.sendFile(req.params.filename, options, (err) => console.log(err))
res.sendFile(req.params.filename, { maxAge: 0, root: path.join(__dirname, "upload") }, (err) => console.log(err))
})
|
|
For more information, please read the Code Analysis documentation
Identify code vulnerabilities directly in yourVS Code editor
Identify code vulnerabilities directly inJetBrains products