Enhancement: make `SessionMiddleware` return custom types about litestar HOT 3 OPEN

since we would have to add another generic parameter to Request.

I'd like to push back a little bit on this. Request[UserT, AuthT, StateT] is already not great UX, IMO. Maybe, once middleware can participate in DI, the need to access these things via a Request object goes away??

Agreed. I really like the idea of being able to inject the session as part of DI. Then we should just hold back on this until v3.

from litestar.

since we would have to add another generic parameter to Request.

I'd like to push back a little bit on this. Request[UserT, AuthT, StateT] is already not great UX, IMO. Maybe, once middleware can participate in DI, the need to access these things via a Request object goes away??

from litestar.

Maybe, once middleware can participate in DI, the need to access these things via a Request object goes away??

Yes, that's more reasonable. Injecting sessions from e.g. a session-plugin that provides the necessary middlewares/DI providers seems more practical.

from litestar.