Skip to content

AST-5032 - Security Fix: Added capability and admin checks to prevent unauthorized font deletion.#149

Merged
yssambare12 merged 7 commits into
masterfrom
security-issue-resolution
Jan 19, 2026
Merged

AST-5032 - Security Fix: Added capability and admin checks to prevent unauthorized font deletion.#149
yssambare12 merged 7 commits into
masterfrom
security-issue-resolution

Conversation

@yssambare12
Copy link
Copy Markdown
Contributor

  • Security Fix: Added capability and admin checks to prevent unauthorized font deletion

Copilot AI review requested due to automatic review settings January 14, 2026 07:48
@yssambare12 yssambare12 requested a review from a team January 14, 2026 07:51
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 pull request implements a security fix to prevent unauthorized font deletion by adding capability and admin area checks to the BCF_Google_Fonts_Compatibility class constructor. The PR also bumps the plugin version from 2.1.16 to 2.1.17.

Changes:

  • Added security checks (is_admin() and current_user_can('manage_options')) before destructive font operations
  • Updated version number across multiple files to 2.1.17
  • Reorganized dependency structures in package-lock.json files

Reviewed changes

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

Show a summary per file
File Description
includes/class-bcf-google-fonts-compatibility.php Added capability and admin area checks to prevent unauthorized font deletion in constructor
custom-fonts.php Updated plugin version to 2.1.17
package.json Updated version to 2.1.17
package-lock.json Updated version (inconsistently) and reorganized prettier dependency location
admin/dashboard/package-lock.json Reorganized string-width-cjs, strip-ansi-cjs, and wrap-ansi-cjs dependencies from nested to root level
README.md Updated stable tag to 2.1.17 and removed trailing spaces from metadata lines
Files not reviewed (1)
  • admin/dashboard/package-lock.json: Language not supported

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

@yssambare12 yssambare12 changed the title CF-001 - Security Fix: Added capability and admin checks to prevent unauthorized font deletion. AST-5023 - Security Fix: Added capability and admin checks to prevent unauthorized font deletion. Jan 15, 2026
@yssambare12 yssambare12 changed the title AST-5023 - Security Fix: Added capability and admin checks to prevent unauthorized font deletion. AST-5032 - Security Fix: Added capability and admin checks to prevent unauthorized font deletion. Jan 15, 2026
@yssambare12 yssambare12 merged commit 7b4e05d into master Jan 19, 2026
2 checks passed
@pratikchaskar pratikchaskar deleted the security-issue-resolution branch May 25, 2026 06:17
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