Always use absolute img url

capy/src/main/java/com/jocmp/capy/Article.kt

@@ -20,5 +20,6 @@ data class Article(
     val feedName: String = "",
     val faviconURL: String? = null,
     val feedURL: String? = null,
+    val siteURL: String? = null,
     val enableStickyFullContent: Boolean = false,
 )

capy/src/main/java/com/jocmp/capy/accounts/LocalAccountDelegate.kt

@@ -13,7 +13,6 @@ import com.jocmp.capy.persistence.TaggingRecords
 import com.jocmp.feedfinder.DefaultFeedFinder
 import com.jocmp.feedfinder.FeedFinder
 import com.prof18.rssparser.model.RssItem
-import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.coroutineScope
 import kotlinx.coroutines.flow.firstOrNull
 import kotlinx.coroutines.launch
@@ -153,7 +152,7 @@ class LocalAccountDelegate(
                 val updated = updatedAt.toEpochSecond()
 
                 val publishedAt = item.pubDate?.toDateTime?.toEpochSecond() ?: updated
-                val url = cleanedArticleLink(item.link, feed.siteURL)
+                val url = cleanedURL(item.link, feed.siteURL)
                 val withinCutoff = cutoffDate == null || publishedAt > cutoffDate.toEpochSecond()
 
                 if (url != null && withinCutoff) {
@@ -166,7 +165,7 @@ class LocalAccountDelegate(
                         url = url.toString(),
                         summary = item.summary,
                         extracted_content_url = null,
-                        image_url = item.image,
+                        image_url = cleanedURL(item.image, siteURL = feed.siteURL)?.toString(),
                         published_at = publishedAt,
                     )
 
@@ -238,8 +237,8 @@ internal val RssItem.summary: String?
         return Jsoup.parse(it).text()
     }
 
-internal fun cleanedArticleLink(articleURL: String?, siteURL: String): URL? {
-    val url = articleURL.orEmpty()
+internal fun cleanedURL(inputURL: String?, siteURL: String): URL? {
+    val url = inputURL.orEmpty()
 
     if (url.isBlank()) {
         return null

capy/src/main/java/com/jocmp/capy/articles/ArticleRenderer.kt

@@ -42,7 +42,7 @@ class ArticleRenderer(
         ).renderedText
 
         val document = Jsoup.parse(html).apply {
-            article.feedURL?.let { setBaseUri(it) }
+            article.siteURL?.let { setBaseUri(it) }
         }
 
         cleanLinks(document)
@@ -53,6 +53,7 @@ class ArticleRenderer(
     private fun cleanLinks(document: Document) {
         document.getElementsByTag("img").forEach { element ->
             element.attr("loading", "lazy")
+            element.attr("src", element.absUrl("src"))
         }
 
         document.select("img[data-src]").forEach { element ->

capy/src/main/java/com/jocmp/capy/common/OptionalURL.kt

@@ -3,7 +3,7 @@ package com.jocmp.capy.common
 import java.net.MalformedURLException
 import java.net.URL
 
-fun optionalURL(string: String?): URL? {
+fun optionalURL(string: String?, baseURL: String? = null): URL? {
     if (string.isNullOrBlank()) {
         return null
     }

capy/src/main/java/com/jocmp/capy/persistence/ArticleMapper.kt

@@ -20,6 +20,7 @@ internal fun articleMapper(
     faviconURL: String?,
     enableStickyContent: Boolean = false,
     feedURL: String?,
+    siteURL: String?,
     updatedAt: Long?,
     starred: Boolean?,
     read: Boolean?,
@@ -35,6 +36,7 @@ internal fun articleMapper(
         url = URL(url),
         imageURL = imageURL,
         feedURL = feedURL,
+        siteURL = siteURL,
         summary = summary ?: "",
         updatedAt = updatedAt!!.toDateTimeFromSeconds,
         publishedAt = publishedAt!!.toDateTimeFromSeconds,
@@ -70,6 +72,7 @@ internal fun listMapper(
         contentHtml = "",
         extractedContentURL = extractedContentURL,
         feedURL = null,
+        siteURL = null,
         url = url,
         feedTitle = feedTitle,
         imageURL = imageURL,

capy/src/main/java/com/jocmp/capy/persistence/ArticleRecords.kt

@@ -276,7 +276,7 @@ internal class ArticleRecords internal constructor(
         }
     }
 
-    internal fun cutoffDate(): ZonedDateTime {
+    private fun cutoffDate(): ZonedDateTime {
         return nowUTC().minusMonths(3)
     }
 }

capy/src/main/sqldelight/com/jocmp/capy/db/articles.sq

@@ -126,6 +126,7 @@ SELECT
   feeds.favicon_url,
   feeds.enable_sticky_full_content,
   feeds.feed_url,
+  feeds.site_url,
   article_statuses.updated_at,
   article_statuses.starred,
   article_statuses.read

capy/src/test/java/com/jocmp/capy/accounts/LocalAccountDelegateTest.kt

@@ -285,7 +285,8 @@ class LocalAccountDelegateTest {
         override val name: String,
         override val feedURL: URL,
         override val siteURL: URL? = null,
-        override val faviconURL: URL? = null
+        override val faviconURL: URL? = null,
+        override val items: List<RssItem> = listOf()
     ) : Feed {
         override fun isValid() = true
     }
@@ -294,42 +295,42 @@ class LocalAccountDelegateTest {
     fun cleanedArticleLink_whenPresent() {
         val url = "https://example.com/article"
 
-        val result = cleanedArticleLink(articleURL = url, siteURL = "")
+        val result = cleanedURL(inputURL = url, siteURL = "")
 
         assertEquals(expected = url, actual = result.toString())
     }
 
     @Test
     fun cleanedArticleLink_whenNull() {
-        val result = cleanedArticleLink(articleURL = null, siteURL = "")
+        val result = cleanedURL(inputURL = null, siteURL = "")
 
         assertEquals(expected = null, actual = result)
     }
 
     @Test
     fun cleanedArticleLink_whenBlank() {
-        val result = cleanedArticleLink(articleURL = "", siteURL = "")
+        val result = cleanedURL(inputURL = "", siteURL = "")
 
         assertEquals(expected = null, actual = result)
     }
 
     @Test
     fun cleanedArticleLink_withRelativePathMissingSiteURL() {
-        val result = cleanedArticleLink(articleURL = "/article", siteURL = "")
+        val result = cleanedURL(inputURL = "/article", siteURL = "")
 
         assertEquals(expected = null, actual = result)
     }
 
     @Test
     fun cleanedArticleLink_withRelativePathAndInvalidSiteURL() {
-        val result = cleanedArticleLink(articleURL = "/article", siteURL = "wrong")
+        val result = cleanedURL(inputURL = "/article", siteURL = "wrong")
 
         assertEquals(expected = null, actual = result)
     }
 
     @Test
     fun cleanedArticleLink_withRelativePathAndValidSiteURL() {
-        val result = cleanedArticleLink(articleURL = "/article", siteURL = "https://example.com")
+        val result = cleanedURL(inputURL = "/article", siteURL = "https://example.com")
 
         assertEquals(expected = "https://example.com/article", actual = result.toString())
     }