fix(cache control): remove unnecessary cache control header from php.conf

[ma-04]

Remove the unnecessary cache control header from the php.conf file. The header is only required for FastCGI cache-enabled sites.

fixes: https://github.com/flywp/issues/issues/183

[coderabbitai]

Walkthrough

The Cache-Control header directive for PHP requests has been removed from the nginx configuration. This prevents the response from explicitly setting cache control parameters, allowing default behavior or other directives to take precedence.

Changes

Cohort / File(s)	Summary
Cache-Control header removal nginx/common/php.conf	Removed the Cache-Control header setting (Cache-Control: public, max-age=3600) from PHP response directives

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• feat: add Cache-Control headers for static and PHP files #7: Inverse operation that adds the Cache-Control header to nginx/common/php.conf for PHP responses, directly related to this removal.

Poem

🐰 A cache header fades away,
No more max-age to hold the day,
The rabbit hops through configs light,
Removing rules to set things right! 🌙

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "fix(cache control): remove unnecessary cache control header from php.conf" directly and accurately describes the main change in the changeset. According to the raw summary, the modification removes the Cache-Control header for PHP requests in nginx/common/php.conf, which is exactly what the title conveys. The title is specific and clear, avoiding vague terminology, and a teammate reviewing the repository history would immediately understand the primary purpose of this change.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[ma-04]

@coderabbitai generate docstrings

[coderabbitai]

✅ Actions performed

Initiated docstring generation; will generate only if new commits exist.

[coderabbitai]

Caution

Docstrings generation - FAILED

No docstrings were generated.