From 7f304f45772bc5652262bfcae97599be9d3d36d2 Mon Sep 17 00:00:00 2001 From: Kisaragi Marine Date: Sun, 28 Apr 2024 11:29:41 +0900 Subject: [PATCH 1/7] =?UTF-8?q?fix(backend):=20=E3=83=8E=E3=83=BC=E3=83=88?= =?UTF-8?q?=E3=81=AEattachment=E3=81=AB=E3=81=8A=E3=81=84=E3=81=A6?= =?UTF-8?q?=E3=80=81attach.sensitive=E3=81=8C=E5=85=83=E3=81=8B=E3=82=89?= =?UTF-8?q?=E5=AD=98=E5=9C=A8=E3=81=99=E3=82=8B=E5=A0=B4=E5=90=88=E3=81=AF?= =?UTF-8?q?=E3=81=9D=E3=82=8C=E3=82=92=E5=B0=8A=E9=87=8D=E3=81=99=E3=82=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../backend/src/core/activitypub/models/ApNoteService.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/packages/backend/src/core/activitypub/models/ApNoteService.ts b/packages/backend/src/core/activitypub/models/ApNoteService.ts index 05f7879983d7..6cabe039df03 100644 --- a/packages/backend/src/core/activitypub/models/ApNoteService.ts +++ b/packages/backend/src/core/activitypub/models/ApNoteService.ts @@ -211,7 +211,12 @@ export class ApNoteService { const files: MiDriveFile[] = []; for (const attach of toArray(note.attachment)) { - attach.sensitive ||= note.sensitive; // Noteがsensitiveなら添付もsensitiveにする + // もしattachment.sensitiveが存在するならそれに従う。なければnote.sensitiveに従う。 + // もしattach.sensitive === falseなのにnote.sensitiveを参照してしまうと、 + // note.sensitiveが立っているMisskey[^1]から来たノートにおいて、 + // リモートサーバーでsensitiveではないと明示されているsensitiveではないファイルもsensitiveとして連合されてしまい、不都合。 + // [^1]: NoteRenderServiceでCWが設定されていたりセンシティブなファイルが存在したりする状況でnote.sensitiveをtrueにする + attach.sensitive ??= note.sensitive; const file = await this.apImageService.resolveImage(actor, attach); if (file) files.push(file); } From de0dabfa097158a41cc91ad8105533ec8b2e4dd7 Mon Sep 17 00:00:00 2001 From: Kisaragi Marine Date: Sun, 28 Apr 2024 11:39:43 +0900 Subject: [PATCH 2/7] docs: update changelog (per misskey-dev#13756) --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 68015596bdcc..aa6bcf2db1d9 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -74,6 +74,7 @@ - Fix: グローバルタイムラインで返信が表示されないことがある問題を修正 - Fix: リノートをミュートしたユーザの投稿のリノートがミュートされる問題を修正 - Fix: AP Link等は添付ファイル扱いしないようになど (#13754) +- Fix: ファイルのセンシティブ設定がリモートサーバーの設定を尊重するように (#13756) ## 2024.3.1 From f167c59538ec1a4c94cc4241076f2f93452ee41b Mon Sep 17 00:00:00 2001 From: Kisaragi Marine Date: Sun, 28 Apr 2024 12:08:50 +0900 Subject: [PATCH 3/7] =?UTF-8?q?feat(backend,apub):=20renderDocument?= =?UTF-8?q?=E3=81=8Csensitive=E3=82=92=E9=80=A3=E5=90=88=E3=81=99=E3=82=8B?= =?UTF-8?q?=E3=82=88=E3=81=86=E3=81=AB=E3=81=99=E3=82=8B=20per=20https://g?= =?UTF-8?q?ithub.com/misskey-dev/misskey/issues/13755#issuecomment-2081303?= =?UTF-8?q?014?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- packages/backend/src/core/activitypub/ApRendererService.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/backend/src/core/activitypub/ApRendererService.ts b/packages/backend/src/core/activitypub/ApRendererService.ts index d7fb977a99d3..f19b3cc2e355 100644 --- a/packages/backend/src/core/activitypub/ApRendererService.ts +++ b/packages/backend/src/core/activitypub/ApRendererService.ts @@ -166,6 +166,7 @@ export class ApRendererService { mediaType: file.webpublicType ?? file.type, url: this.driveFileEntityService.getPublicUrl(file), name: file.comment, + sensitive: file.isSensitive, }; } From 1b69c0f820e7992c8353607ca2630aa992512b2f Mon Sep 17 00:00:00 2001 From: Kisaragi <48310258+KisaragiEffective@users.noreply.github.com> Date: Sun, 28 Apr 2024 12:43:03 +0900 Subject: [PATCH 4/7] =?UTF-8?q?chore(backend):=20=E8=BF=BD=E5=8A=A0?= =?UTF-8?q?=E3=81=97=E3=81=9F=E3=82=B3=E3=83=A1=E3=83=B3=E3=83=88=E3=82=92?= =?UTF-8?q?=E5=89=8A=E9=99=A4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../backend/src/core/activitypub/models/ApNoteService.ts | 5 ----- 1 file changed, 5 deletions(-) diff --git a/packages/backend/src/core/activitypub/models/ApNoteService.ts b/packages/backend/src/core/activitypub/models/ApNoteService.ts index 6cabe039df03..4e361b57bcf5 100644 --- a/packages/backend/src/core/activitypub/models/ApNoteService.ts +++ b/packages/backend/src/core/activitypub/models/ApNoteService.ts @@ -211,11 +211,6 @@ export class ApNoteService { const files: MiDriveFile[] = []; for (const attach of toArray(note.attachment)) { - // もしattachment.sensitiveが存在するならそれに従う。なければnote.sensitiveに従う。 - // もしattach.sensitive === falseなのにnote.sensitiveを参照してしまうと、 - // note.sensitiveが立っているMisskey[^1]から来たノートにおいて、 - // リモートサーバーでsensitiveではないと明示されているsensitiveではないファイルもsensitiveとして連合されてしまい、不都合。 - // [^1]: NoteRenderServiceでCWが設定されていたりセンシティブなファイルが存在したりする状況でnote.sensitiveをtrueにする attach.sensitive ??= note.sensitive; const file = await this.apImageService.resolveImage(actor, attach); if (file) files.push(file); From c26299e300ee80a2947ad77969ba9e0b76a21a05 Mon Sep 17 00:00:00 2001 From: Kisaragi <48310258+KisaragiEffective@users.noreply.github.com> Date: Sun, 28 Apr 2024 12:43:35 +0900 Subject: [PATCH 5/7] =?UTF-8?q?docs:=20changelog=E3=82=92=E3=82=88?= =?UTF-8?q?=E3=82=8A=E4=B8=81=E5=AF=A7=E3=81=AB=E3=81=99=E3=82=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index aa6bcf2db1d9..ee9e8f00322a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -75,6 +75,8 @@ - Fix: リノートをミュートしたユーザの投稿のリノートがミュートされる問題を修正 - Fix: AP Link等は添付ファイル扱いしないようになど (#13754) - Fix: ファイルのセンシティブ設定がリモートサーバーの設定を尊重するように (#13756) + - ドライブに存在するファイルがNSFWかどうか連合されるようになります + - 可能な場合、ファイルのセンシティブ判定がファイル単位になります ## 2024.3.1 From 934214ae25d71485a9ce7f538defd18fe6c128f6 Mon Sep 17 00:00:00 2001 From: Kisaragi <48310258+KisaragiEffective@users.noreply.github.com> Date: Sun, 28 Apr 2024 12:49:15 +0900 Subject: [PATCH 6/7] =?UTF-8?q?docs:=20changelog=E3=81=AE=E9=A0=85?= =?UTF-8?q?=E7=9B=AE=E5=90=8D=E3=82=92PR=E3=81=AB=E5=90=88=E3=82=8F?= =?UTF-8?q?=E3=81=9B=E3=82=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- CHANGELOG.md | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index ee9e8f00322a..7549178d6fd2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -74,9 +74,8 @@ - Fix: グローバルタイムラインで返信が表示されないことがある問題を修正 - Fix: リノートをミュートしたユーザの投稿のリノートがミュートされる問題を修正 - Fix: AP Link等は添付ファイル扱いしないようになど (#13754) -- Fix: ファイルのセンシティブ設定がリモートサーバーの設定を尊重するように (#13756) - - ドライブに存在するファイルがNSFWかどうか連合されるようになります - - 可能な場合、ファイルのセンシティブ判定がファイル単位になります +- Enhance: ドライブのファイルがNSFWかどうか連合されるように (#13756) + - 可能な場合、ノートの添付ファイルのセンシティブ判定がファイル単位になります ## 2024.3.1 From aac5618d0ad98149e5c74db7fc1890b5df85eb04 Mon Sep 17 00:00:00 2001 From: Kisaragi <48310258+KisaragiEffective@users.noreply.github.com> Date: Sun, 28 Apr 2024 12:59:29 +0900 Subject: [PATCH 7/7] docs: tweak apply suggestion from mei23 --- CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 7549178d6fd2..8345138a4898 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -74,7 +74,7 @@ - Fix: グローバルタイムラインで返信が表示されないことがある問題を修正 - Fix: リノートをミュートしたユーザの投稿のリノートがミュートされる問題を修正 - Fix: AP Link等は添付ファイル扱いしないようになど (#13754) -- Enhance: ドライブのファイルがNSFWかどうか連合されるように (#13756) +- Enhance: ドライブのファイルがNSFWかどうか個別に連合されるように (#13756) - 可能な場合、ノートの添付ファイルのセンシティブ判定がファイル単位になります ## 2024.3.1