Fixed issue wit Symfony6 BC break

php-pm / php-pm-httpkernel

HttpKernel adapter for use of Symfony and Laravel frameworks with PHP-PM

MIT License

246 stars 72 forks source link

Fixed issue wit Symfony6 BC break #184

Closed KonstantinPopov closed 2 years ago

KonstantinPopov commented 2 years ago

Symfony Bootstraps used HttpFoundation\Request constants for booting application. in Symfony6 was removed deprecated const Symfony\Component\HttpFoundation\Request::HEADER_X_FORWARDED_ALL was replaced on. HEADER_X_FORWARDED_FOR | HEADER_X_FORWARDED_HOST | HEADER_X_FORWARDED_PORT | HEADER_X_FORWARDED_PROTO as described on symfony recommendation.

andig commented 2 years ago

I wouldn‘t know. Any objections?

KonstantinPopov commented 2 years ago

I wouldn‘t know. Any objections?

https://github.com/symfony/http-foundation/blob/5.4/Request.php#L52 in Symfony 6 this const was removed. so after update receive error

mathieudz commented 2 years ago

@andig Fine with me, please merge. I need it too.