WPIntell

Source evidence

esc_html on $value_array and $valueAr

Debug Meta Data · support · 2016-08-26T04:12:00+00:00

complaintsentiment
mediumseverity
0.75relevance
1replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

6 / 35 rows with source links

17.1% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

29 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
Jesse Graupmann unresolved
First off, thanks for this plugin. Its great. Now I just noticed something that may/may not be by design. Essentially when you output the content of meta, you’re not escaping the value. For cases where your meta can contain script tags they will execute on the edit screen. Consider the following: foreach ( $value as $name_array => $value_array ) { echo $name_array . ' => ' . esc_html( $value_array ); echo var_dump( $value_array ); } Without the esc_html , the following immediately redirects you to another page. If this was on a server I don’t know a lot of people that would be able to modify and stop the code from executing. <script> window.location = "http://www.example.com"; </script> If this is by design, it is what it is. But if you’re not opposed to escaping the output I would highly recommend it. Thanks! https://wordpress.org/plugins/debug-meta-data/ It was an intentional decision by design. This plugin is strictly meant to be used over localhost. What do you think? What I can do is add a settings page to disable it and have two enable options with or with escaped data?

Comments

1 shown
Ahmad Awais 2016-09-15T17:30:00+00:00

It was an intentional decision by design. This plugin is strictly meant to be used over localhost. What do you think? What I can do is add a settings page to disable it and have two enable options with or with escaped data?