Skip to content

Dark theme and icon to toggle theme#682

Merged
digininja merged 4 commits intodigininja:masterfrom
eoto88:dark-theme
Apr 2, 2025
Merged

Dark theme and icon to toggle theme#682
digininja merged 4 commits intodigininja:masterfrom
eoto88:dark-theme

Conversation

@eoto88
Copy link
Copy Markdown
Contributor

@eoto88 eoto88 commented Apr 2, 2025

Dark theme perfect for late night hacking. Icon is from Material Design : https://pictogrammers.com/library/mdi/

@eoto88
Copy link
Copy Markdown
Contributor Author

eoto88 commented Apr 2, 2025

Attempt number 2 😛

Comment thread dvwa/includes/dvwaPage.inc.php Outdated
@digininja
Copy link
Copy Markdown
Owner

Can you get a tooltip on the icon you've added to say what it does?

@eoto88
Copy link
Copy Markdown
Contributor Author

eoto88 commented Apr 2, 2025

Tooltip and injection vulnerability from cookie added.


function dvwaThemeGet() {
if (isset($_COOKIE['theme'])) {
return $_COOKIE[ 'theme' ];

Check failure

Code scanning / PHP Security Analysis

Detected tainted HTML: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\ncall to echo. Error

Detected tainted HTML: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\ncall to echo.

function dvwaThemeGet() {
if (isset($_COOKIE['theme'])) {
return $_COOKIE[ 'theme' ];

Check failure

Code scanning / PHP Security Analysis

Detected tainted text with possible quotes: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\ncall to echo. Error

Detected tainted text with possible quotes: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\ncall to echo.

function dvwaThemeGet() {
if (isset($_COOKIE['theme'])) {
return $_COOKIE[ 'theme' ];

Check failure

Code scanning / PHP Security Analysis

Detected tainted HTML: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\ncall to echo. Error

Detected tainted HTML: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\ncall to echo.

function dvwaThemeGet() {
if (isset($_COOKIE['theme'])) {
return $_COOKIE[ 'theme' ];

Check failure

Code scanning / PHP Security Analysis

Detected tainted text with possible quotes: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\ncall to echo. Error

Detected tainted text with possible quotes: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\ncall to echo.

function dvwaThemeGet() {
if (isset($_COOKIE['theme'])) {
return $_COOKIE[ 'theme' ];

Check failure

Code scanning / PHP Security Analysis

Detected tainted HTML: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\ncall to echo. Error

Detected tainted HTML: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\ncall to echo.

function dvwaThemeGet() {
if (isset($_COOKIE['theme'])) {
return $_COOKIE[ 'theme' ];

Check failure

Code scanning / PHP Security Analysis

Detected tainted text with possible quotes: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\nconcat\ncall to echo. Error

Detected tainted text with possible quotes: $_COOKIE['theme']\ndvwaThemeGet\nconcat\nconcat\ncall to echo.
@digininja digininja merged commit c5cf03a into digininja:master Apr 2, 2025
5 of 6 checks passed
@digininja
Copy link
Copy Markdown
Owner

GitHub code scanner doesn't like it, I'm happy with that!

image

@eoto88 eoto88 deleted the dark-theme branch April 9, 2025 00:53
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Dark theme and icon to toggle theme
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Dark theme and icon to toggle theme
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Dark theme and icon to toggle theme
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Dark theme and icon to toggle theme
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Dark theme and icon to toggle theme
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Dark theme and icon to toggle theme
noe-orga-NTT pushed a commit to noe-orga-NTT/DVWA that referenced this pull request May 30, 2025
Dark theme and icon to toggle theme
BrunoCascante pushed a commit to BrunoCascante/DVWA that referenced this pull request Jun 16, 2025
Dark theme and icon to toggle theme
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.

3 participants