Skip to content

Added a vulnerable API module#670

Merged
digininja merged 31 commits into
masterfrom
api
Jan 29, 2025
Merged

Added a vulnerable API module#670
digininja merged 31 commits into
masterfrom
api

Conversation

@digininja

Copy link
Copy Markdown
Owner

No description provided.

@digininja digininja merged commit a96943d into master Jan 29, 2025
$client_secret = $_SERVER['PHP_AUTH_PW'];

# App auth check
if ($client_id == "1471.dvwa.digi.ninja" && $client_secret == "ABigLongSecret") {

Check failure

Code scanning / Secrets Audit

Cleartext Storage of Sensitive Information.

Credential in plaintext? Rule: Env Var Line: if ($client_id == "1471.dvwa.digi.ninja" && $client_secret == "ABigLongSecret") { Commit: .
}

// pass the request method and order ID to the OrderController and process the HTTP request:
$controller = new OrderController($requestMethod, $version, $orderId);

Check warning

Code scanning / PHP Security Audit

Class Src \ OrderController has no __construct, but arguments were passed.

Class Src \ OrderController has no __construct, but arguments were passed.
}

// pass the request method and user ID to the UserController and process the HTTP request:
$controller = new UserController($requestMethod, $version, $userId);

Check warning

Code scanning / PHP Security Audit

Class Src \ OrderController has no __construct, but arguments were passed.

Class Src \ UserController has no __construct, but arguments were passed.
}

$command = $local_uri[2];
$controller = new HealthController($requestMethod, $version, $command);

Check warning

Code scanning / PHP Security Audit

Class Src \ OrderController has no __construct, but arguments were passed.

Class Src \ HealthController has no __construct, but arguments were passed.
}

$command = $local_uri[2];
$controller = new LoginController($requestMethod, $version, $command);

Check warning

Code scanning / PHP Security Audit

Class Src \ OrderController has no __construct, but arguments were passed.

Class Src \ LoginController has no __construct, but arguments were passed.
}

#[OAT\Post(
tags: ["health"],

Check warning

Code scanning / PHP Security Audit

Syntax error, unexpected T_STRING on line 22.

Syntax error, unexpected T_STRING on line 22.
}

#[OAT\Get(
tags: ["user"],

Check warning

Code scanning / PHP Security Audit

Syntax error, unexpected T_STRING on line 22.

Syntax error, unexpected T_STRING on line 63.
}

#[OAT\Get(
tags: ["user"],

Check warning

Code scanning / PHP Security Audit

Syntax error, unexpected T_STRING on line 22.

Syntax error, unexpected T_STRING on line 98.
}

#[OAT\Post(
tags: ["user"],

Check warning

Code scanning / PHP Security Audit

Syntax error, unexpected T_STRING on line 22.

Syntax error, unexpected T_STRING on line 126.
}

#[OAT\Put(
tags: ["user"],

Check warning

Code scanning / PHP Security Audit

Syntax error, unexpected T_STRING on line 22.

Syntax error, unexpected T_STRING on line 175.
}

#[OAT\Delete(
tags: ["user"],

Check warning

Code scanning / PHP Security Audit

Syntax error, unexpected T_STRING on line 22.

Syntax error, unexpected T_STRING on line 233.
@digininja digininja deleted the api branch February 26, 2025 09:53
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Added a vulnerable API module
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Added a vulnerable API module
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Added a vulnerable API module
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Added a vulnerable API module
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Added a vulnerable API module
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Added a vulnerable API module
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Added a vulnerable API module
BrunoCascante pushed a commit to BrunoCascante/DVWA that referenced this pull request Jun 16, 2025
Added a vulnerable API module
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