Clarification: 1.3.6 "dangerous characters"
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 3.6k
- Forks
- 831
- Avg merge
- 7h 55m
- Merged PRs (30d)
- 4
Description
Verify that the application protects against Server‑side Request Forgery
(SSRF) attacks, by validating untrusted data against an allowlist of protocols,
domains, paths and ports and sanitizing potentially dangerous characters
before using the data to call another service.
"Sanitizing Potentially Dangerous Characters"
- What characters are considered as "dangerous'?
- Why sanitization is important? I am having difficulty find the information about sanitization for SSRF, I have referered Below URL:
| Reference Site | Comment |
|---|---|
| Cheatsheet | Missing Sanitization Word |
| OWASP TOP 10 SSRF | Found 'Sanitize' word only. |
| Portswigger | Missing Sanitization Word |
| Accunetix | Missing Sanitization Word |
| SSRF Bible Cheatsheet | Missing Sanitization Word |
| Blackhat | Missing Sanitization Word |
As far as I know, the sanitization is not the primary defense. the requirement doesn’t specify how granular or strict the allow list should be and also other important terms are missing in the requirement.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read the referenced ASVS 5.0 requirement and its raw Markdown source, then compare the linked SSRF references and the discussion in this issue. Determine whether the wording should define dangerous characters, explain sanitization, or clarify the requirement's scope; done means an agreed clarification is reflected in the requirement text.
Written by the indexing model from the issue text.
Assessment
- Domain
- documentation, security
- Issue type
- Documentation
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100