php xdebug log location

[adamoutler]

📌 Description

Updates the Xdebug log path in the PHP configuration to use /tmp/log/ instead of /app/log/ to ensure proper write permissions within the container environment.

---

🔍 Related Issues

---

📋 Type of Change

• 🐛 Bug fix
• 🔧 Build/config update

---

📷 Screenshots or Logs (if applicable)

Was: xdebug.log=/app/log/xdebug.log
Is: xdebug.log=/tmp/log/xdebug.log

---

🧪 Testing Steps

1. Rebuild the devcontainer.
2. Trigger an Xdebug session.
3. Verify log creation at /tmp/log/xdebug.log.

---

✅ Checklist

• I have read the Contribution Guidelines
• I have tested my changes locally
• I have updated relevant documentation (if applicable)
• I have verified my changes do not break existing behavior
• I am willing to respond to requested changes and feedback

---

🙋 Additional Notes

The /app/log directory was inappropriate; /tmp/log/ is the proper location.

Summary by CodeRabbit

• Chores
  • Updated development environment logging configuration to use an alternative log file location.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The Xdebug configuration in the development container was updated to redirect the log file location from /app/log/xdebug.log to /tmp/log/xdebug.log. This is a configuration-only adjustment with no functional behavioral changes.

Changes

Cohort / File(s)	Summary
Xdebug Configuration .devcontainer/resources/devcontainer-overlay/services/config/php/conf.d/99-xdebug.ini	Updated xdebug.log path from /app/log/xdebug.log to /tmp/log/xdebug.log

Poem

🐰 A log hops from app to tmp so clean,
Where Xdebug's traces are freshly seen,
In dev containers, debugging's delight,
The path now flows to temporal light! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: updating the PHP Xdebug log location from /app/log/xdebug.log to /tmp/log/xdebug.log.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}