Skip to content

Commit 3848a0b

Browse files
committed
Merge pull request #248 from dodok8/dodok8-fix-issu-246
fix: Direct messages are leaked on public post pages
2 parents a37cfb6 + e8a07f7 commit 3848a0b

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

CHANGES.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,14 @@ Version 0.6.12
66

77
To be released.
88

9+
- Fixed a critical security vulnerability where direct messages were leaked
10+
on public post pages. The replies list below posts now correctly filters
11+
to show only public or unlisted replies, preventing private conversations
12+
from being exposed. [[#246], [#248] by Hyeonseo Kim]
13+
14+
[#246]: https://github.com/fedify-dev/hollo/issues/246
15+
[#248]: https://github.com/fedify-dev/hollo/pull/248
16+
917

1018
Version 0.6.11
1119
--------------

src/pages/profile/profilePost.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { and, eq, or } from "drizzle-orm";
1+
import { and, eq, inArray, or } from "drizzle-orm";
22
import { Hono } from "hono";
33
import { Layout } from "../../components/Layout.tsx";
44
import { Post as PostView } from "../../components/Post.tsx";
@@ -66,6 +66,7 @@ profilePost.get<"/:handle{@[^/]+}/:id{[-a-f0-9]+}">(async (c) => {
6666
},
6767
},
6868
replies: {
69+
where: inArray(posts.visibility, ["public", "unlisted"]),
6970
with: {
7071
account: true,
7172
media: true,

0 commit comments

Comments
 (0)