Pavel Kirilin

Results 228 comments of Pavel Kirilin
trafficstars

Yes. The problem is that FastAPI framework uses types to inject dependencies. Quoting "Redis[Any]" doesn't work. Importing `from __future__ import annotations` also doesn't work. Here is a minimum reproducible example:...

But maybe, if I provide a request with Redis type without Generic in definition, it will work as expected. Or it would be incorrect?

Ok. I'll keep this issue open to be able to link pull request to it. Can you assign this issue to me?

The code I attached to the pull request is an MRE.

I replaced default allocator and it helped a lot. In production we don't have memory leaks now.

I think it's irrelevant. Since testing works well.

@andcarnivorous Hi. Thanks for your contribution! Just open the pr, and we'll discuss everything within it.

Hi, @vmjcv. The table should follow this protocol: https://github.com/fastapi-users/fastapi-users/blob/ff9fae631cdae00ebc15f051e54728b3c8d11420/fastapi_users/models.py#L6. Anyway, I consider this as a problem, for sure. We need to fix that. Thanks noticing.

I will take a look today. Maybe there was a problem specific to the setup.