dunglas / frankenphp

🧟 The modern PHP app server
https://frankenphp.dev
MIT License
6.96k stars 244 forks source link

Fix: empty request headers. #1182

Closed Alliballibaba2 closed 1 day ago

Alliballibaba2 commented 1 day ago

This PR fixes empty request headers as mentioned in #1181. I originally though this was a by-product of #1106, but that PR just enforces the wrong behavior.

The problem comes from the fact that pinning an empty string makes the string NULL instead of empty (something to keep in mind).

dunglas commented 1 day ago

Thank you!