OWASP / OWASP/ASVS

Clarification: 1.3.6 "dangerous characters"

Open
#3,244 25 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

4a) Waiting for another V1 (prev V5)
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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.