[PR #5762] [MERGED] Fix network test on dns failure #10643

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/5762
Author: @BaronGreenback
Created: 4/10/2021
Status: Merged
Merged: 5/5/2021
Merged by: @thornbill

Base: masterHead: networkTestHardening


📝 Commits (2)

  • 1fe26fe Work through dns failure on test.
  • 10d358c Update tests/Jellyfin.Networking.Tests/NetworkParseTests.cs

📊 Changes

1 file changed (+8 additions, -2 deletions)

View changed files

📝 tests/Jellyfin.Networking.Tests/NetworkParseTests.cs (+8 -2)

📄 Description

Test shouldn't now fail in the case of dns resolution failure.


🔄 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/5762 **Author:** [@BaronGreenback](https://github.com/BaronGreenback) **Created:** 4/10/2021 **Status:** ✅ Merged **Merged:** 5/5/2021 **Merged by:** [@thornbill](https://github.com/thornbill) **Base:** `master` ← **Head:** `networkTestHardening` --- ### 📝 Commits (2) - [`1fe26fe`](https://github.com/jellyfin/jellyfin/commit/1fe26fe352a854523337842b16d6d0be5e68be5a) Work through dns failure on test. - [`10d358c`](https://github.com/jellyfin/jellyfin/commit/10d358c8da4f5e323c81a5880625737e025d2ba0) Update tests/Jellyfin.Networking.Tests/NetworkParseTests.cs ### 📊 Changes **1 file changed** (+8 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `tests/Jellyfin.Networking.Tests/NetworkParseTests.cs` (+8 -2) </details> ### 📄 Description Test shouldn't now fail in the case of dns resolution failure. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:24:07 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10643