Add video proxy to fix Firefox tracking protection blocking Twitter videos
Twitter's video.twimg.com CDN is blocked by Firefox's Enhanced Tracking
Protection when loaded cross-origin. This adds a server-side proxy that
streams videos through our domain.
- Add /proxy/video/{id} endpoint to proxy embed video URLs
- Update embed video player: click-to-play with overlay, muted, looping
- Show native controls on hover
This commit is contained in:
parent
2887d9c430
commit
9aa8373055
5 changed files with 130 additions and 2 deletions
|
|
@ -53,9 +53,12 @@ func (c itemPageContent) Render(sw *ssr.Writer) error {
|
|||
{{else if eq .Item.ItemType "embed"}}
|
||||
{{if .Item.EmbedVideoURL}}
|
||||
<div class="video-container">
|
||||
<video controls poster="{{if .Item.ThumbnailID}}/media/{{.Item.ThumbnailID}}{{end}}">
|
||||
<source src="{{.Item.EmbedVideoURL}}" type="video/mp4">
|
||||
<video loop muted playsinline poster="{{if .Item.ThumbnailID}}/media/{{.Item.ThumbnailID}}{{end}}">
|
||||
<source src="/proxy/video/{{.Item.ID}}" type="video/mp4">
|
||||
</video>
|
||||
<div class="video-overlay" onclick="playVideo(this)">
|
||||
<span class="play-button">▶</span>
|
||||
</div>
|
||||
</div>
|
||||
{{else if .Item.ThumbnailID}}
|
||||
<div class="image-container">
|
||||
|
|
|
|||
77
internal/handlers/proxy.go
Normal file
77
internal/handlers/proxy.go
Normal file
|
|
@ -0,0 +1,77 @@
|
|||
package handlers
|
||||
|
||||
import (
|
||||
"context"
|
||||
"io"
|
||||
"net/http"
|
||||
"time"
|
||||
|
||||
"lookbook/internal/data/item"
|
||||
)
|
||||
|
||||
// HandleProxyVideo proxies video requests for embed items to avoid
|
||||
// cross-origin issues with external video CDNs (e.g., Twitter's video.twimg.com).
|
||||
// GET /proxy/video/{id}
|
||||
func HandleProxyVideo(rc *RequestContext, w http.ResponseWriter, r *http.Request) error {
|
||||
pubID := r.PathValue("id")
|
||||
if pubID == "" {
|
||||
http.NotFound(w, r)
|
||||
return nil
|
||||
}
|
||||
|
||||
ctx, cancel := context.WithTimeout(r.Context(), 30*time.Second)
|
||||
defer cancel()
|
||||
|
||||
it, err := item.QFindByPubID(ctx, rc.DB, pubID)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
if it == nil || !it.EmbedVideoURL.Valid || it.EmbedVideoURL.V == "" {
|
||||
http.NotFound(w, r)
|
||||
return nil
|
||||
}
|
||||
|
||||
videoURL := it.EmbedVideoURL.V
|
||||
|
||||
req, err := http.NewRequestWithContext(ctx, "GET", videoURL, nil)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
|
||||
req.Header.Set("User-Agent", "Mozilla/5.0 (compatible; Lookbook/1.0)")
|
||||
|
||||
// Pass through Range header for video seeking
|
||||
if rangeHeader := r.Header.Get("Range"); rangeHeader != "" {
|
||||
req.Header.Set("Range", rangeHeader)
|
||||
}
|
||||
|
||||
client := &http.Client{Timeout: 2 * time.Minute}
|
||||
resp, err := client.Do(req)
|
||||
if err != nil {
|
||||
http.Error(w, "Failed to fetch video", http.StatusBadGateway)
|
||||
return nil
|
||||
}
|
||||
defer resp.Body.Close()
|
||||
|
||||
// Pass through relevant headers
|
||||
if ct := resp.Header.Get("Content-Type"); ct != "" {
|
||||
w.Header().Set("Content-Type", ct)
|
||||
}
|
||||
if cl := resp.Header.Get("Content-Length"); cl != "" {
|
||||
w.Header().Set("Content-Length", cl)
|
||||
}
|
||||
if cr := resp.Header.Get("Content-Range"); cr != "" {
|
||||
w.Header().Set("Content-Range", cr)
|
||||
}
|
||||
if ar := resp.Header.Get("Accept-Ranges"); ar != "" {
|
||||
w.Header().Set("Accept-Ranges", ar)
|
||||
}
|
||||
|
||||
// Cache for 1 day in the browser
|
||||
w.Header().Set("Cache-Control", "public, max-age=86400")
|
||||
|
||||
w.WriteHeader(resp.StatusCode)
|
||||
io.Copy(w, resp.Body)
|
||||
|
||||
return nil
|
||||
}
|
||||
|
|
@ -296,6 +296,40 @@ button, input, textarea, select {
|
|||
width: 100%;
|
||||
}
|
||||
|
||||
.video-container {
|
||||
position: relative;
|
||||
}
|
||||
|
||||
.video-overlay {
|
||||
position: absolute;
|
||||
inset: 0;
|
||||
display: flex;
|
||||
align-items: center;
|
||||
justify-content: center;
|
||||
background: rgba(0, 0, 0, 0.3);
|
||||
cursor: pointer;
|
||||
transition: background 0.15s ease;
|
||||
}
|
||||
|
||||
.video-overlay:hover {
|
||||
background: rgba(0, 0, 0, 0.4);
|
||||
}
|
||||
|
||||
.video-overlay .play-button {
|
||||
width: 4rem;
|
||||
height: 4rem;
|
||||
background: rgba(0, 0, 0, 0.7);
|
||||
color: #fff;
|
||||
border-radius: 50%;
|
||||
display: flex;
|
||||
align-items: center;
|
||||
justify-content: center;
|
||||
font-size: 1.5rem;
|
||||
padding-left: 0.25rem;
|
||||
}
|
||||
|
||||
|
||||
|
||||
.embed-container {
|
||||
position: relative;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -378,6 +378,19 @@ async function deleteItem(id) {
|
|||
}
|
||||
}
|
||||
|
||||
// Play video (initial click to start)
|
||||
function playVideo(overlay) {
|
||||
const container = overlay.parentElement;
|
||||
const video = container.querySelector('video');
|
||||
if (video) {
|
||||
video.play();
|
||||
overlay.remove();
|
||||
// Show controls on hover
|
||||
video.addEventListener('mouseenter', () => video.controls = true);
|
||||
video.addEventListener('mouseleave', () => video.controls = false);
|
||||
}
|
||||
}
|
||||
|
||||
// Utility
|
||||
function escapeHtml(str) {
|
||||
const div = document.createElement('div');
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue