Skip to content

Add ability to set transport DSN directly to default transport factory. #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
May 15, 2017
Merged

Add ability to set transport DSN directly to default transport factory. #81

merged 3 commits into from
May 15, 2017

Conversation

makasim
Copy link
Member

@makasim makasim commented May 12, 2017 •
edited
Loading

The idea is to have a factory that internally resolves what transport to use. (ref symfony/recipes-contrib#16)

enqueue:
  transport:
    default: 'null://'
#    default:  'file:/%kernel.root_dir%/../var/enqueue'
#    default: 'amqp://'

@makasim makasim changed the base branch from amqp-dsn to master May 12, 2017 19:52
@makasim makasim changed the title Revert "Revert "[symfony] dsn transport factory."" [PoC] DsnTransportFactory May 12, 2017
makasim added 2 commits May 15, 2017 10:09
@makasim
Merge branch 'master' into dsn-transport
a7b4ab0 
pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php
@makasim
add ability to guess a factory by dsn scheme to default transport fac…
553974a 
…tory.
@makasim makasim changed the title [PoC] DsnTransportFactory Add ability to set transport DSN directly to default transport factory. May 15, 2017
@makasim makasim requested a review from ASKozienko May 15, 2017 11:18
@makasim makasim merged commit 1c6d2a9 into master May 15, 2017
@makasim makasim deleted the dsn-transport branch May 15, 2017 11:19
ASKozienko pushed a commit that referenced this pull request Nov 2, 2018
@makasim
Merge pull request #81 from php-enqueue/dsn-transport
bf83b0c 
Add ability to set transport DSN directly to default transport factory.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ASKozienko ASKozienko Awaiting requested review from ASKozienko

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@makasim