aljaz/boris
1
1
Fork 0
mirror of https://github.com/dergigi/boris.git synced 2026-02-23 07:54:59 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: improve URL regex patterns to prevent text artifacts

Browse Source 
- Updated VideoEmbedProcessor regex patterns to use lookahead assertions
- This prevents capturing HTML attribute syntax like quotes and angle brackets
- Fixes text artifact appearing in UI when processing video URLs in HTML content
This commit is contained in:
Gigi
2025-10-20 20:45:22 +02:00
parent 64aad42be3
commit a5494ba15c
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/components/VideoEmbedProcessor.tsx
View File

@@ -31,7 +31,8 @@ const VideoEmbedProcessor = forwardRef<HTMLDivElement, VideoEmbedProcessorProps>
const videoUrls: string[] = html.match(videoUrlPattern) || []
// Also check for video URLs that might not have extensions but are classified as video
const allUrlPattern = /https?:\/\/[^\s<>"']+/gi
// Use a more precise pattern that stops at whitespace, quotes, and HTML tag boundaries
const allUrlPattern = /https?:\/\/[^\s<>"']+(?=\s|>|"|'|$)/gi
const allUrls: string[] = html.match(allUrlPattern) || []
const videoUrlsWithoutExt = allUrls.filter(url => {
const classification = classifyUrl(url)
@@ -62,7 +63,8 @@ const VideoEmbedProcessor = forwardRef<HTMLDivElement, VideoEmbedProcessorProps>
const videoUrlPattern = /https?:\/\/[^\s<>"']+\.(mp4|webm|ogg|mov|avi|mkv|m4v)(?:\?[^\s<>"']*)?/gi
const videoUrls: string[] = html.match(videoUrlPattern) || []
const allUrlPattern = /https?:\/\/[^\s<>"']+/gi
// Use a more precise pattern that stops at whitespace, quotes, and HTML tag boundaries
const allUrlPattern = /https?:\/\/[^\s<>"']+(?=\s|>|"|'|$)/gi
const allUrls: string[] = html.match(allUrlPattern) || []
const videoUrlsWithoutExt = allUrls.filter(url => {
const classification = classifyUrl(url)