Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 25, 2025

  • Created new branch from php8 base (as requested by @yama)
  • Applied SQL injection fix
  • Fixed where() function to escape values using db()->escape()
  • Changed to single quotes for SQL string literals

This is a rebased version of the previous PR, now based on the php8 branch as requested.

Original prompt

This section details on the original issue you should resolve

<issue_title>$keyはエスケープが必要</issue_title>
<issue_description>

where('cache_key', '=', $key)

db()->escape($key)</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

yama and others added 3 commits August 1, 2022 10:44
# Conflicts:
#	assets/snippets/ditto/extenders/example2.extender.inc.php
@yama
Copy link
Member

yama commented Dec 25, 2025

@copilot php8ブランチからブランチを切る必要があります

Copilot AI and others added 2 commits December 25, 2025 04:16
Co-authored-by: yama <73999+yama@users.noreply.github.com>
Co-authored-by: yama <73999+yama@users.noreply.github.com>
Copilot AI changed the title [WIP] Escape $key in document parser Fix SQL injection in where() helper by escaping values Dec 25, 2025
Copilot AI requested a review from yama December 25, 2025 04:20
Copy link
Contributor Author

Copilot AI commented Dec 25, 2025

@copilot php8ブランチからブランチを切る必要があります

ブランチをphp8から切り直しました。変更をコミット cbf9f07 で適用しました。

ただし、report_progressツールでは force push ができないため、リモートブランチの更新に問題が発生しています。PRを閉じて新しいブランチから再作成する必要があるかもしれません。

@yama
Copy link
Member

yama commented Dec 25, 2025

@copilot php8ブランチからブランチを切る必要があります

ブランチをphp8から切り直しました。変更をコミット cbf9f07 で適用しました。

ただし、report_progressツールでは force push ができないため、リモートブランチの更新に問題が発生しています。PRを閉じて新しいブランチから再作成する必要があるかもしれません。

そのようです。再作成してください。

@yama yama closed this Dec 25, 2025
@yama yama deleted the copilot/escape-key-in-document-parser branch December 25, 2025 04:33
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.

$keyはエスケープが必要

2 participants