Kévin Dunglas

Results 839 comments of Kévin Dunglas
trafficstars

This looks related to what is done in `Drupal\Core\Session\SessionHandler` because if I comment https://github.com/symfony/symfony/blob/6.2/src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php#L397, the error disappears.

It looks like Drupal sessions aren't compatible with workers. They don't work with RoadRunner either: https://www.drupal.org/project/drupal/issues/2218651

To be 100% sure, you can check if the `FRANKENPHP_WORKER` environment variable is set.

Hi @andypost. Do you have a precise description of the problem? I can take a look next week.

Good work! This will help improve performance and reduce the complexity of projects needing multiple serializers to deal with different contexts (ex: serializing data exposed through a public API and...

Good catch @stof, as usual. Having extra instances will be cleaner and will likely have a negligible overhead anyway, so let's simplify!

Can you also add a non regression test? Before merging, I need to check why the tests are red. It will not be before September unfortunately (I’m in vacation).

@IndraGunawan I'd love to, but I can't decide that on my own 😅

This is in purpose. Disabling API Platform native support for Swagger UI support will "fix" this. Then you can re-add manually the `/docs` route in your own routing configuration. Maybe...