[PR #9935] [MERGED] Fix Websocket OpenApi #12101

Closed
opened 2025-12-22 09:06:39 +01:00 by backuprepo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/9935
Author: @crobibero
Created: 6/27/2023
Status: Merged
Merged: 6/29/2023
Merged by: @crobibero

Base: masterHead: ws-openapi


📝 Commits (3)

  • db9cc8f Further split inbound and outbound messages
  • 4c33d65 Fix datatype for inbound start messages
  • 651eacc fixes from review

📊 Changes

50 files changed (+165 additions, -126 deletions)

View changed files

📝 Emby.Server.Implementations/HttpServer/WebSocketConnection.cs (+14 -2)
📝 Emby.Server.Implementations/Session/WebSocketController.cs (+2 -2)
📝 MediaBrowser.Controller/Net/BasePeriodicWebSocketListener.cs (+2 -2)
📝 MediaBrowser.Controller/Net/IWebSocketConnection.cs (+0 -1)
📝 MediaBrowser.Controller/Net/WebSocketMessage.cs (+0 -6)
📝 MediaBrowser.Controller/Net/WebSocketMessageInfo.cs (+0 -2)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Inbound/ActivityLogEntryStartMessage.cs (+5 -5)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Inbound/ActivityLogEntryStopMessage.cs (+1 -12)
MediaBrowser.Controller/Net/WebSocketMessages/Inbound/InboundKeepAliveMessage.cs (+14 -0)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Inbound/ScheduledTasksInfoStartMessage.cs (+4 -5)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Inbound/ScheduledTasksInfoStopMessage.cs (+1 -12)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Inbound/SessionsStartMessage.cs (+4 -4)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Inbound/SessionsStopMessage.cs (+1 -11)
📝 MediaBrowser.Controller/Net/WebSocketMessages/InboundWebSocketMessage.cs (+3 -4)
MediaBrowser.Controller/Net/WebSocketMessages/InboundWebSocketMessageOfT.cs (+26 -0)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Outbound/ActivityLogEntryMessage.cs (+1 -1)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Outbound/ForceKeepAliveMessage.cs (+1 -1)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Outbound/GeneralCommandMessage.cs (+1 -1)
📝 MediaBrowser.Controller/Net/WebSocketMessages/Outbound/LibraryChangedMessage.cs (+1 -1)
MediaBrowser.Controller/Net/WebSocketMessages/Outbound/OutboundKeepAliveMessage.cs (+14 -0)

...and 30 more files

📄 Description

Changes

Issues

Fixes #9934

OpenAPI spec: openapi.10.9.json.txt


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/9935 **Author:** [@crobibero](https://github.com/crobibero) **Created:** 6/27/2023 **Status:** ✅ Merged **Merged:** 6/29/2023 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `ws-openapi` --- ### 📝 Commits (3) - [`db9cc8f`](https://github.com/jellyfin/jellyfin/commit/db9cc8f794c37b035b0e295db9df1c3fa24697ba) Further split inbound and outbound messages - [`4c33d65`](https://github.com/jellyfin/jellyfin/commit/4c33d65e5f41c097c5206d479acff17465932e38) Fix datatype for inbound start messages - [`651eacc`](https://github.com/jellyfin/jellyfin/commit/651eacc0d1c1d894af5c81d7129911a6459ecd25) fixes from review ### 📊 Changes **50 files changed** (+165 additions, -126 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/HttpServer/WebSocketConnection.cs` (+14 -2) 📝 `Emby.Server.Implementations/Session/WebSocketController.cs` (+2 -2) 📝 `MediaBrowser.Controller/Net/BasePeriodicWebSocketListener.cs` (+2 -2) 📝 `MediaBrowser.Controller/Net/IWebSocketConnection.cs` (+0 -1) 📝 `MediaBrowser.Controller/Net/WebSocketMessage.cs` (+0 -6) 📝 `MediaBrowser.Controller/Net/WebSocketMessageInfo.cs` (+0 -2) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Inbound/ActivityLogEntryStartMessage.cs` (+5 -5) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Inbound/ActivityLogEntryStopMessage.cs` (+1 -12) ➕ `MediaBrowser.Controller/Net/WebSocketMessages/Inbound/InboundKeepAliveMessage.cs` (+14 -0) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Inbound/ScheduledTasksInfoStartMessage.cs` (+4 -5) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Inbound/ScheduledTasksInfoStopMessage.cs` (+1 -12) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Inbound/SessionsStartMessage.cs` (+4 -4) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Inbound/SessionsStopMessage.cs` (+1 -11) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/InboundWebSocketMessage.cs` (+3 -4) ➕ `MediaBrowser.Controller/Net/WebSocketMessages/InboundWebSocketMessageOfT.cs` (+26 -0) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Outbound/ActivityLogEntryMessage.cs` (+1 -1) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Outbound/ForceKeepAliveMessage.cs` (+1 -1) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Outbound/GeneralCommandMessage.cs` (+1 -1) 📝 `MediaBrowser.Controller/Net/WebSocketMessages/Outbound/LibraryChangedMessage.cs` (+1 -1) ➕ `MediaBrowser.Controller/Net/WebSocketMessages/Outbound/OutboundKeepAliveMessage.cs` (+14 -0) _...and 30 more files_ </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** <!-- Describe your changes here in 1-5 sentences. --> **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> Fixes #9934 OpenAPI spec: [openapi.10.9.json.txt](https://github.com/jellyfin/jellyfin/files/11882516/openapi.10.9.json.txt) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
backuprepo 2025-12-22 09:06:39 +01:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#12101
No description provided.