Skip to content

Add unscoped object lookup IDOR test case - #12

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:feat/add-idor-object-access-javascript
Aug 4, 2026
Merged

Add unscoped object lookup IDOR test case#12
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:feat/add-idor-object-access-javascript

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Test Case Submission

  • I have read docs/CONTRIBUTING.md
  • This test case is non-destructive — no disk, network, or process side effects
  • Every payload is behind a dead-code guard (if (false))
  • Every host literal uses a reserved domain (this case contains no host literals)
  • Any credential is a published example or dead placeholder (this case contains no credentials)
  • I have included all required metadata
  • I marked the expected line with VULNERABLE: and the control with SAFE:
  • I ran scripts/validate-test-case.sh and it passed
  • I regenerated the catalog and committed the result

The local Windows environment has no Bash. I ran the script's six equivalent checks individually: metadata, dead-code guards, network destinations, root manifests, catalog freshness, and Python/JSON syntax. I also ran Node syntax validation and git diff --check; all passed.

Details

  • Vulnerability type: Insecure direct object reference via unscoped lookup
  • CWE ID: CWE-639
  • Language: JavaScript
  • Expected detection: Yes
  • Why this pattern matters: Authenticated handlers often trust route identifiers without constraining database lookups to the current owner. This creates a data-flow authorization flaw that scanners should distinguish from a correctly owner-scoped query.

Safety statement

Every lookup and response is inside a literal if (false) block. The database helper is a local placeholder that only returns inert JavaScript objects. The file performs no network, disk, process, credential, or external database operation.

Scanner result

Leave blank on submission — CI fills this in.

@ralyodio
ralyodio merged commit 12afa51 into profullstack:master Aug 4, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants