Skip to content

Fix/security part6#1086

Merged
obiPlabon merged 7 commits intoalphafrom
fix/security-part6
Aug 2, 2022
Merged

Fix/security part6#1086
obiPlabon merged 7 commits intoalphafrom
fix/security-part6

Conversation

@vairafiq
Copy link
Contributor

@vairafiq vairafiq commented Aug 2, 2022

No description provided.

Copy link
Contributor

@kowsar89 kowsar89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


FILE: includes/helper-functions.php
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 7155 | ERROR | All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$html'.
      |       | (WordPress.Security.EscapeOutput.OutputNotEscaped)
 8027 | ERROR | All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$the_html'.
      |       | (WordPress.Security.EscapeOutput.OutputNotEscaped)
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Time: 485ms; Memory: 64.01MB


@kowsar89
Copy link
Contributor

kowsar89 commented Aug 2, 2022

includes/class-helper.php includes/custom-actions.php includes/helper-functions.php includes/template-functions.php

@obiPlabon obiPlabon merged commit 63b7a0f into alpha Aug 2, 2022
@obiPlabon obiPlabon deleted the fix/security-part6 branch August 2, 2022 10:16
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