Skip to content
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.

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

[Symfony] add support to transfer tokenStorage (user info) to the worker #69

Closed
dkarlovi opened this issue May 4, 2017 · 1 comment
Closed
Labels
enhancement pr needed closed bugs or features issues that require a PR. Because there is noone is willig to contribute it

Comments

@dkarlovi
Copy link
Contributor

dkarlovi commented May 4, 2017

Implementing #68, I've found the processor needs to have information which user ran the query (I'm using the Blameable Doctrine2 extension).

I've added this as a temporary hack in the implementation, but the real solution would be to have an extension which would transfer the current tokenStorage from the message origin to the processor. This way we could use other Symfony features (like authorizationChecker for example) in our processors as naturally as we would in the web app.

We coul probably also trigger some core Symfony event (like kernel.request?) to make other listeners run, but that would be a nice bonus: after having the user token in place, it's easy doing some of the stuff yourself.

@makasim
Copy link
Member

makasim commented Nov 15, 2018

PR is welcome

@makasim makasim closed this as completed Nov 15, 2018
@makasim makasim added the pr needed closed bugs or features issues that require a PR. Because there is noone is willig to contribute it label May 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement pr needed closed bugs or features issues that require a PR. Because there is noone is willig to contribute it
Projects
None yet
Development

No branches or pull requests

2 participants