fix(backend): Fix an issue where notes are still visible over Feed even if the user required login | ノートの閲覧にログイン必須にしてもFeedでノートが表示されてしまう問題を修正

[eternal-flame-AD]

What

ユーザーがログインしたユーザーのみに閲覧を制限している場合でも、メモはRSS経由で読むことができる問題を修正しました。

Sample RSS

<rss version="2.0" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:content="http://purl.org/rss/1.0/modules/content/">
    <channel>
        <title>admin (@admin@localhost:5173)</title>
        <link>http://localhost:5173/@admin</link>
        <description>3 Notes, 0 Following, 0 Followers</description>
        <lastBuildDate>Mon, 02 Dec 2024 02:29:03 GMT</lastBuildDate>
        <docs>https://validator.w3.org/feed/docs/rss2.html</docs>
        <generator>Misskey</generator>
        <image>
            <title>admin (@admin@localhost:5173)</title>
            <url>http://localhost:5173/identicon/admin@localhost:5173</url>
            <link>http://localhost:5173/@admin</link>
        </image>
        <copyright>admin</copyright>
        <item>
            <title><![CDATA[New note by admin]]></title>
            <link>http://localhost:5173/notes/a19z78harr7t0009</link>
            <guid>http://localhost:5173/notes/a19z78harr7t0009</guid>
            <pubDate>Mon, 02 Dec 2024 02:29:03 GMT</pubDate>
            <content:encoded><![CDATA[<p>test 4aa9e7aa89a2261d83b2056e4a</p>]]></content:encoded>
        </item>
        <item>
            <title><![CDATA[New note by admin]]></title>
            <link>http://localhost:5173/notes/a0xew8tv6la30005</link>
            <guid>http://localhost:5173/notes/a0xew8tv6la30005</guid>
            <pubDate>Sat, 23 Nov 2024 07:27:23 GMT</pubDate>
        </item>
    </channel>
</rss>⏎ 

Why

元のディスカッション (#14473) では、この機能はクローラー防止に使用されと述べられている。しかしRSS はクローラーによって広くサポートされているため、これは予期しない動作だと思います。

Additional info (optional)

Checklist

• Read the contribution guide
• Test working in a local environment
• (If needed) Add story of storybook
• (If needed) Update CHANGELOG.md
• (If possible) Add tests

[github-actions]

このPRによるapi.jsonの差分
差分はありません。
Get diff files from Workflow Page

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 38.75%. Comparing base (f4bca47) to head (9aad74a).
Report is 2 commits behind head on develop.

Files with missing lines	Patch %	Lines
...ages/backend/src/server/web/ClientServerService.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #15083      +/-   ##
===========================================
- Coverage    38.75%   38.75%   -0.01%     
===========================================
  Files         1601     1601              
  Lines       204015   204016       +1     
  Branches      4012     4012              
===========================================
  Hits         79065    79065              
- Misses      124314   124315       +1     
  Partials       636      636              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mi-gh-maintainer]

Thank you 🙏