search

repman-io / repman

Repman - PHP Repository Manager: packagist proxy and host for private packages
https://repman.io
MIT License
515 stars 107 forks source link

Allow other symfony/messenger transports than doctrine #533

Open dheineman opened 2 years ago

dheineman commented 2 years ago

The repman environment file has a section for symfony/messenger to allow you to choose a transport by configuring a DSN. https://github.com/repman-io/repman/blob/174c38190086e9ebf1f6d7849469742c8edd4e68/.env#L41-L46

However the symfony/messenger config yaml has the doctrine transport hardcoded instead of using this environment variable https://github.com/repman-io/repman/blob/174c38190086e9ebf1f6d7849469742c8edd4e68/config/packages/messenger.yaml#L10-L12

Similarly only the symfony/doctrine-messenger is required in the composer.json file. https://github.com/repman-io/repman/blob/174c38190086e9ebf1f6d7849469742c8edd4e68/composer.json#L58

It there any specific reason this was hardcoded to doctrine? I would like to use the redis transport as it allows multiple workers consuming the same queue.

akondas commented 2 years ago

It there any specific reason this was hardcoded to doctrine?

No, and I don't see any obstacles to change it :wink: