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

add param inject for auto inject middlewares #237

Open
IvanKirpichnikov opened this issue Aug 31, 2024 · 3 comments
Open

add param inject for auto inject middlewares #237

IvanKirpichnikov opened this issue Aug 31, 2024 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@IvanKirpichnikov
Copy link
Member

It will be possible to pass your own injection to this parameter. The default injection will be default.
This is necessary in order not to have identical middleware in your code with the difference of different injection decorators

@Tishka17
Copy link
Member

Tishka17 commented Sep 3, 2024

@IvanKirpichnikov we need details here

@IvanKirpichnikov
Copy link
Member Author

I implemented my thoughts in #238

@IvanKirpichnikov
Copy link
Member Author

I am faced with the problem that my inject decorators analyze kw-only parameters. Due to the fact that the standard auto inject integrations use the default inject, I have to duplicate the code because of this.

This problem can be solved by adding a parameter that accepts the injection function and uses it in the future.

@Tishka17 Tishka17 added the enhancement New feature or request label Sep 13, 2024
@Tishka17 Tishka17 added this to the 1.5 milestone Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants