Skip to content

AST-5003 - Astra Widget Security Issue. - #76

Merged
yssambare12 merged 3 commits into
masterfrom
security-fix
Dec 15, 2025
Merged

AST-5003 - Astra Widget Security Issue.#76
yssambare12 merged 3 commits into
masterfrom
security-fix

Conversation

@yssambare12

Copy link
Copy Markdown
Contributor
  • Astra Widget Security Issue.

Copilot AI review requested due to automatic review settings December 10, 2025 06:18
@yssambare12
yssambare12 requested a review from a team December 10, 2025 06:18

Copilot AI left a comment

Copy link
Copy Markdown

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 a security vulnerability in the Astra Widgets plugin by implementing proper output escaping and input sanitization across three widget classes. The version is bumped from 1.2.16 to 1.2.17.

  • Replaced esc_attr() with esc_url() for URL output in href attributes to prevent XSS attacks
  • Added esc_url_raw() sanitization in widget update methods to sanitize user-provided URLs during save
  • Changed esc_attr() to esc_html() for text content display to use the appropriate escaping function

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
readme.txt Updated stable tag to 1.2.17 and added changelog entry for security improvements
package.json Bumped version to 1.2.17
package-lock.json Bumped version to 1.2.17
astra-widgets.php Updated plugin version header to 1.2.17
README.md Updated stable tag to 1.2.17 and added changelog entry for security improvements
classes/widgets/class-astra-widget-social-profiles.php Changed link output escaping from esc_attr() to esc_url() and added esc_url_raw() sanitization in the update method for social profile links
classes/widgets/class-astra-widget-list-icons.php Added esc_url_raw() sanitization in the update method for list icon links after the generic text field sanitization
classes/widgets/class-astra-widget-address.php Changed address display to use esc_html(), and updated telephone and email href attributes to use esc_url() instead of building with esc_attr(), plus changed fax display to use esc_html()

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

@yssambare12
yssambare12 merged commit ee41313 into master Dec 15, 2025
7 checks passed
@yssambare12
yssambare12 deleted the security-fix branch December 15, 2025 06:23
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