Bref Symfony Bridge recipe

[ker0x]

Bref Symfony Bridge recipe

[ker0x]

I'm closing the issue as #1129 has been merged. Thanks to everyone for your advices and reviews ❤️

[fabpot]

We accept recipes for any packages available on Packagist (we do have plenty on non-bundles already).

Replacing an existing file is not an option as it would make debugging/upgrading/... much harder. We should find another way. Maybe the new runtime component can help here?

Generating files is one of the things a recipe can do, so I don't any issue here.

[mnapoli]

@OskarStark that is the same amount of changes as telling the user to extend Bref's modified Kernel.

[OskarStark]

Ok I thought it would need more adjustments 👍🏻

[t-richard]

Thanks everyone for the input ! @ker0x I think we should start simple

1. copy-from-recipe for the serverless.yaml
2. add .serverless in gitignore
3. add a post-install.txt to explain what to do with the kernel

Once done, we could see what more can be done with the Kernel.

What do you think ?

[mnapoli]

@fabpot thank you for the answer!

Replacing an existing file is not an option as it would make debugging/upgrading/... much harder. We should find another way. Maybe the new runtime component can help here?

Nyholm is working hard on that, the runtime component could be useful in some ways, but AFAIK not directly for this problem.

On Lambda, logs & cache need to be moved elsewhere (because the filesystem is read-only except for /tmp). The only way to do that is by editing Kernel.php (unless there's another way?).

Worst case scenario: we ask the user to do the change manually, which isn't the end of the world. But I'd love to know if there's an alternative that could be automated.

[OskarStark]

What about adding a ServerlessKernel.php file next to the Kernel and tell the user to change one line in index.php ?

[t-richard]

What about adding a ServerlessKernel.php file next to the Kernel and tell the user to change one line in index.php ?

IMHO it's just moving the manual action to another place.

The current documented way with the Kernel is below and moving to index.php would be similar.

// src/Kernel.php

namespace App;

+ use Bref\SymfonyBridge\BrefKernel;
use Symfony\Bundle\FrameworkBundle\Kernel\MicroKernelTrait;
use Symfony\Component\Config\Loader\LoaderInterface;
use Symfony\Component\Config\Resource\FileResource;
use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\HttpKernel\Kernel as BaseKernel;
use Symfony\Component\Routing\RouteCollectionBuilder;

- class Kernel extends BaseKernel
+ class Kernel extends BrefKernel
{
    // ...

[mnapoli]

@t-richard sounds good, though I wouldn't put any documentation file in the project: readers should be following the documentation already so we can explain the last step there.

[ker0x]

@t-richard it sounds good to me too, I will start to work on it this week probably !

@mnapoli At first the post-install.txt file will only show a link to the documentation (which will need to be updated though). We can always improve it later

[mnapoli]

@ker0x I'd rather not have a documentation file inserted into the project, that doesn't correspond to any practice found in other open-source projects. Users are already following the documentation, we can instruct them on what to do inside the documentation.

[mnapoli]

Thank you @ker0x!