Skip to content

Panels: escape output of Debug_Bar_WP_Query class.#20

Merged
JJJ merged 1 commit intotrunkfrom
issue-19
Mar 1, 2026
Merged

Panels: escape output of Debug_Bar_WP_Query class.#20
JJJ merged 1 commit intotrunkfrom
issue-19

Conversation

@JJJ
Copy link
Copy Markdown
Contributor

@JJJ JJJ commented Mar 1, 2026

This includes a specific fix provided via H1 - thanks, b4shu206!

Fixes #19.

This includes a specific fix provided via H1 - thanks, b4shu206!

Fixes #19.
Copilot AI review requested due to automatic review settings March 1, 2026 19:43
@JJJ JJJ merged commit be7cec5 into trunk Mar 1, 2026
2 checks passed
@JJJ JJJ deleted the issue-19 branch March 1, 2026 19:44
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses Issue #19 by escaping HTML output in the Debug_Bar_WP_Query panel class to prevent potential XSS vulnerabilities. All dynamic values passed to echo in the render() method and the _recursive_print_kv() helper method are now wrapped with esc_html().

Changes:

  • Dynamic values in render() (Queried Object ID, Query Type, Show on Front, Page for Posts, Page on Front, Post Type) are now escaped with esc_html().
  • Keys and scalar values in _recursive_print_kv() are now escaped with esc_html().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JJJ
Copy link
Copy Markdown
Contributor Author

JJJ commented Mar 2, 2026

Pull request overview

This PR addresses Issue #19 by escaping HTML output in the Debug_Bar_WP_Query panel class to prevent potential XSS vulnerabilities. All dynamic values passed to echo in the render() method and the _recursive_print_kv() helper method are now wrapped with esc_html().

Changes:

  • Dynamic values in render() (Queried Object ID, Query Type, Show on Front, Page for Posts, Page on Front, Post Type) are now escaped with esc_html().
  • Keys and scalar values in _recursive_print_kv() are now escaped with esc_html().

This was done here: c7bad83

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.

Escape output of _recursive_print_kv()

2 participants