[xExtension-ImageProxy] Video poster images are not proxied
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 74/100
Direzione di ricerca
Inizia nell’estensione Image Proxy e cerca dove riscrive gli attributi img, come src e srcset; esamina la gestione correlata del caricamento differito e il modo in cui FreshRSS legge l’URL originale dell’immagine. Estendi questo percorso a poster dei video, mantenendo l’URL originale in data-xextension-imageproxy-original-poster, poi controlla un articolo con un poster video e conferma che src del video rimanga invariato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the bug
The Image Proxy extension rewrites the src and srcset of <img> elements, but not the poster attribute of <video> elements. FreshRSS keeps poster when it sanitises feed content, so for articles with embedded videos the browser loads the poster image straight from the original server. This bypasses the proxy, which is what users with Proxy HTTPS enabled want to avoid for privacy, and it can also cause mixed-content warnings for http:// posters on an HTTPS instance.
To Reproduce
- Enable the Image Proxy extension, with Proxy HTTPS checked.
- Display an article whose content contains
<video src="https://videos.example/a.mp4" poster="https://images.example/poster.jpg"></video>. - The poster is requested from
images.exampleand not through the proxy. In the HTML source,poster(moved todata-originalby the lazy-loading) still points to the original URL.
Expected behavior
Video posters should go through the proxy like other images, with the original URL kept in a data-xextension-imageproxy-original-poster attribute. That way FreshRSS still recognises the URL when deciding whether to append the entry's thumbnail or enclosures, as it already does for <img> elements. The video itself (src) should not be proxied, because image proxies cannot serve it.
FreshRSS version
edge (Image Proxy extension 1.0.2)
- Lingua principale
- PHP
- Stelle
- 731
- Fork
- 120
- Merge medio
- 13h 7m
- PR unite (30g)
- 22
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di FreshRSS/Extensions
-
Feature Request
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
FreshRSS/Extensions#514 · 2 commenti · 3 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
[Webhook] {content} placeholder turns escaped text into HTML markupForse già presa @NopeCharon l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 35/100
FreshRSS/Extensions#560 ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug] shorts stopped being filtered by sept 7.Forse già presa @JamBalaya56562 l’ha presa 30 giorni fa. ApertaBug (unconfirmed) help wanted xExtension-YouTube
Difficoltà 3/5 1-2 giorni Idoneità per principianti 62/100
FreshRSS/Extensions#529 · 12 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
help wanted php
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
FreshRSS/Extensions#525 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
FreshRSS/Extensions#521 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di FreshRSS/Extensions
Issue simili
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
Automattic/safe-publish#764 ·
I maintainer di solito rispondono entro 1 giorno
-
sync-en
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
sync-en
Difficoltà 1/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
Перевод устарел
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
grokability/snipe-it#19941 ·
I maintainer di solito rispondono entro 1 giorno