[PR #3508] Part 1: nullable Emby.DLNA #9567

Closed
opened 2026-02-07 06:04:35 +03:00 by OVERLORD · 0 comments
Owner

Original Pull Request: https://github.com/jellyfin/jellyfin/pull/3508

State: closed
Merged: Yes


Trying to break this down into manageable parts.

If this change is accepted, then there is far less work and less changes to make any IRequest instance nullable.

It makes the Request object available at construction time, so the Request doesn't have to be nullable.

The request object is stored in

httpContextAccessor.HttpContext.Items["ServiceStackRequest"]

and can be accessed via:

public abcService(IHttpContextAccessor httpContextAccessor)
{
        Request = (IRequest)httpContextAccessor.HttpContext.Items["ServiceStackRequest"];
}
**Original Pull Request:** https://github.com/jellyfin/jellyfin/pull/3508 **State:** closed **Merged:** Yes --- Trying to break this down into manageable parts. If this change is accepted, then there is far less work and **less changes** to make any IRequest instance nullable. It makes the Request object available at construction time, so the Request doesn't have to be nullable. The request object is stored in ``` httpContextAccessor.HttpContext.Items["ServiceStackRequest"] ``` and can be accessed via: ``` public abcService(IHttpContextAccessor httpContextAccessor) { Request = (IRequest)httpContextAccessor.HttpContext.Items["ServiceStackRequest"]; } ```
OVERLORD added the pull-request label 2026-02-07 06:04:35 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9567