🔀 Merge pull request #172 from anuarkaliyev23/fix/171-escape-in-title

davep · web-flow · commit 26912315e69c · 2025-09-04T15:57:43.000+01:00
Fixes #171 — Unescaped titles in bookmarks causing braindrop to crash
diff --git a/src/braindrop/app/widgets/raindrop_details.py b/src/braindrop/app/widgets/raindrop_details.py
@@ -265,7 +265,7 @@ def _refresh_display(self) -> None:
             self._set(
                 "collection",
                 f"{PUBLIC_ICON if self.data.collection(self.raindrop.collection).public else PRIVATE_ICON}"
-                f" {self.data.collection(self.raindrop.collection).title}",
+                f" {escape(self.data.collection(self.raindrop.collection).title)}",
             )
             self._set("note", self.raindrop.note, Markdown)
             self._set(

Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,7 @@ def _refresh_display(self) -> None:`
`265`	`265`	`self._set(`
`266`	`266`	`"collection",`
`267`	`267`	`f"{PUBLIC_ICON if self.data.collection(self.raindrop.collection).public else PRIVATE_ICON}"`
`268`		`- f" {self.data.collection(self.raindrop.collection).title}",`
	`268`	`+ f" {escape(self.data.collection(self.raindrop.collection).title)}",`
`269`	`269`	`)`
`270`	`270`	`self._set("note", self.raindrop.note, Markdown)`
`271`	`271`	`self._set(`