diff --git a/internal/reader/rewrite/content_rewrite.go b/internal/reader/rewrite/content_rewrite.go
index 4d806dad..db6db6e2 100644
--- a/internal/reader/rewrite/content_rewrite.go
+++ b/internal/reader/rewrite/content_rewrite.go
@@ -94,6 +94,8 @@ func (rule rule) applyRule(entryURL string, entry *model.Entry) {
entry.Title = titlelize(entry.Title)
case "fix_ghost_cards":
entry.Content = fixGhostCards(entry.Content)
+ case "remove_img_blur_params":
+ entry.Content = removeImgBlurParams(entry.Content)
}
}
@@ -130,7 +132,7 @@ func parseRules(rulesText string) (rules []rule) {
rules[l].args = append(rules[l].args, text)
}
case scanner.EOF:
- return
+ return rules
}
}
}
diff --git a/internal/reader/rewrite/content_rewrite_functions.go b/internal/reader/rewrite/content_rewrite_functions.go
index a3858270..c2551fe7 100644
--- a/internal/reader/rewrite/content_rewrite_functions.go
+++ b/internal/reader/rewrite/content_rewrite_functions.go
@@ -10,6 +10,7 @@ import (
"log/slog"
"net/url"
"regexp"
+ "strconv"
"strings"
"unicode"
@@ -547,3 +548,43 @@ func fixGhostCards(entryContent string) string {
output, _ := doc.FindMatcher(goquery.Single("body")).Html()
return strings.TrimSpace(output)
}
+
+func removeImgBlurParams(entryContent string) string {
+ doc, err := goquery.NewDocumentFromReader(strings.NewReader(entryContent))
+ if err != nil {
+ return entryContent
+ }
+
+ changed := false
+
+ doc.Find("img[src]").Each(func(i int, img *goquery.Selection) {
+ srcAttr, exists := img.Attr("src")
+ if !exists {
+ return
+ }
+
+ parsedURL, err := url.Parse(srcAttr)
+ if err != nil {
+ return
+ }
+
+ // Only strip query parameters if this is a blurry placeholder image
+ if parsedURL.RawQuery != "" {
+ // Check if there's a blur parameter with a non-zero value
+ if blurValue := parsedURL.Query().Get("blur"); blurValue != "" {
+ if blurInt, err := strconv.Atoi(blurValue); err == nil && blurInt > 0 {
+ parsedURL.RawQuery = ""
+ img.SetAttr("src", parsedURL.String())
+ changed = true
+ }
+ }
+ }
+ })
+
+ if changed {
+ output, _ := doc.FindMatcher(goquery.Single("body")).Html()
+ return output
+ }
+
+ return entryContent
+}
diff --git a/internal/reader/rewrite/content_rewrite_test.go b/internal/reader/rewrite/content_rewrite_test.go
index 307dd747..2aa9d6b4 100644
--- a/internal/reader/rewrite/content_rewrite_test.go
+++ b/internal/reader/rewrite/content_rewrite_test.go
@@ -133,7 +133,6 @@ func TestRewriteYoutubeLinkAndCustomEmbedURL(t *testing.T) {
var err error
parser := config.NewConfigParser()
config.Opts, err = parser.ParseEnvironmentVariables()
-
if err != nil {
t.Fatalf(`Parsing failure: %v`, err)
}
@@ -241,7 +240,6 @@ func TestAddYoutubeVideoFromIdWithCustomEmbedURL(t *testing.T) {
var err error
parser := config.NewConfigParser()
config.Opts, err = parser.ParseEnvironmentVariables()
-
if err != nil {
t.Fatalf(`Parsing failure: %v`, err)
}
@@ -797,6 +795,7 @@ func TestRewriteRemoveCustom(t *testing.T) {
t.Errorf(`Not expected output: got "%+v" instead of "%+v"`, testEntry, controlEntry)
}
}
+
func TestRewriteRemoveQuotedSelector(t *testing.T) {
controlEntry := &model.Entry{
URL: "https://example.org/article",
@@ -1248,3 +1247,158 @@ func TestFixGhostCardMultipleSplit(t *testing.T) {
t.Errorf(`Not expected output: got "%+v" instead of "%+v"`, testEntry, controlEntry)
}
}
+
+func TestStripImageQueryParams(t *testing.T) {
+ testEntry := &model.Entry{
+ URL: "https://example.org/article",
+ Title: `News Article Title`,
+ Content: `
+
+ Article content with images having query parameters:
+ 
+ 
+
+ More images with various query parameters:
+ 
+ 
+
+ Image without query parameters:
+ 
+
+ Images with various other params:
+ 
+ 
+ `,
+ }
+
+ controlEntry := &model.Entry{
+ URL: "https://example.org/article",
+ Title: `News Article Title`,
+ Content: `
+ Article content with images having query parameters:
+ 
+
+
+ More images with various query parameters:
+ 
+
+
+ Image without query parameters:
+
+
+ Images with various other params:
+
+
+ `,
+ }
+ ApplyContentRewriteRules(testEntry, `remove_img_blur_params`)
+
+ if !reflect.DeepEqual(testEntry, controlEntry) {
+ t.Errorf(`Not expected output: got "%+v" instead of "%+v"`, testEntry, controlEntry)
+ }
+}
+
+func TestStripImageQueryParamsNoChanges(t *testing.T) {
+ testEntry := &model.Entry{
+ URL: "https://example.org/article",
+ Title: `Article Without Images`,
+ Content: `No images here:
+ Just some text content
+ A link`,
+ }
+
+ controlEntry := &model.Entry{
+ URL: "https://example.org/article",
+ Title: `Article Without Images`,
+ Content: `No images here:
+ Just some text content
+ A link`,
+ }
+ ApplyContentRewriteRules(testEntry, `remove_img_blur_params`)
+
+ if !reflect.DeepEqual(testEntry, controlEntry) {
+ t.Errorf(`Not expected output: got "%+v" instead of "%+v"`, testEntry, controlEntry)
+ }
+}
+
+func TestStripImageQueryParamsEdgeCases(t *testing.T) {
+ testEntry := &model.Entry{
+ URL: "https://example.org/article",
+ Title: `Edge Cases`,
+ Content: `
+ Edge cases for image query parameter stripping:
+
+
+ 
+
+
+ 
+ 
+
+
+ 
+
+
+ 
+ `,
+ }
+
+ controlEntry := &model.Entry{
+ URL: "https://example.org/article",
+ Title: `Edge Cases`,
+ Content: `Edge cases for image query parameter stripping:
+
+
+ 
+
+
+
+
+
+
+ 
+
+
+
+ `,
+ }
+ ApplyContentRewriteRules(testEntry, `remove_img_blur_params`)
+
+ if !reflect.DeepEqual(testEntry, controlEntry) {
+ t.Errorf(`Not expected output: got "%+v" instead of "%+v"`, testEntry, controlEntry)
+ }
+}
+
+func TestStripImageQueryParamsSimple(t *testing.T) {
+ testEntry := &model.Entry{
+ URL: "https://example.org/article",
+ Title: `Simple Test`,
+ Content: `
+ Testing query parameter stripping:
+
+
+ 
+ 
+ 
+ 
+ `,
+ }
+
+ controlEntry := &model.Entry{
+ URL: "https://example.org/article",
+ Title: `Simple Test`,
+ Content: `Testing query parameter stripping:
+
+
+
+ 
+
+
+ `,
+ }
+ ApplyContentRewriteRules(testEntry, `remove_img_blur_params`)
+
+ if !reflect.DeepEqual(testEntry, controlEntry) {
+ t.Errorf(`Not expected output: got "%+v" instead of "%+v"`, testEntry, controlEntry)
+ }
+}