[PR #2104] [MERGED] Simplify regex to avoid catastrophic backtracking #8823

New issue

Closed

opened 2025-12-22 07:08:55 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 07:08:55 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/2104
Author: @cvium
Created: 12/6/2019
Status: ✅ Merged
Merged: 12/6/2019
Merged by: @joshuaboniface

Base: master ← Head: avoid_catastrophic_backtracking

📝 Commits (1)

8befab5 Simplify regex to avoid catastrophic backtracking

📊 Changes

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

View changed files

📝 Emby.Naming/Common/NamingOptions.cs (+2 -2)

📄 Description

The regex used was performing very poorly because of excessive backtracking. Example: https://regex101.com/r/DkpDYz/5

Issues
Hopefully fixes:

#2090
#2092 (maybe)
#2059

_{🔄 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/2104 **Author:** [@cvium](https://github.com/cvium) **Created:** 12/6/2019 **Status:** ✅ Merged **Merged:** 12/6/2019 **Merged by:** [@joshuaboniface](https://github.com/joshuaboniface) **Base:** `master` ← **Head:** `avoid_catastrophic_backtracking` --- ### 📝 Commits (1) - [`8befab5`](https://github.com/jellyfin/jellyfin/commit/8befab5b5dc586a7673a1dc384cf41044c37351c) Simplify regex to avoid catastrophic backtracking ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Naming/Common/NamingOptions.cs` (+2 -2) </details> ### 📄 Description The regex used was performing very poorly because of excessive backtracking. Example: https://regex101.com/r/DkpDYz/5 **Issues** Hopefully fixes: - #2090 - #2092 (maybe) - #2059 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo

2025-12-22 07:08:55 +01:00

closed this issue
added the
pull-request
label

backuprepo referenced this issue

2025-12-22 08:53:44 +01:00

[PR #8823] [MERGED] Do HEAD request to get content type instead of checking for extension #11690

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#8823

No description provided.

Rows
Columns