Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ダイレクト投稿をRNできる #11969

Closed
anatawa12 opened this issue Oct 5, 2023 · 2 comments · Fixed by #11970
Closed

ダイレクト投稿をRNできる #11969

anatawa12 opened this issue Oct 5, 2023 · 2 comments · Fixed by #11970
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR

Comments

@anatawa12
Copy link
Member

anatawa12 commented Oct 5, 2023

(オリジナルは niri-la#56 by @KisaragiEffective)

Originally ダイレクト投稿なRNを削除すると500エラーになる

💡 Summary

投稿範囲をダイレクトにしたノートをRNし、そのRNを取り消すとステータスコードが500のレスポンスとエラーが返ってくる

(※おそらく misskey-dev#10127 と同根のバグだと思われるが、upstreamで発生するか検証していないため一旦このリポジトリに記録)

anatawa12追記: そもそもダイレクト投稿がrenoteできるのが誤りでないかとも思っています

🥰 Expected Behavior

200などで正常に消去される

🤬 Actual Behavior

消去はされているように見えるが、レスポンスでエラーが返ってくる

📝 Steps to Reproduce

  1. 投稿範囲をダイレクトにしたノートをする
  2. そのノートをリノートする
  3. リノートを取り消す

📌 Environment

💻 Frontend

  • Model and OS of the device(s):macOS 14.0 23A344 arm64
  • Browser:firefox 118.0.1 (64 ビット
  • Server URL: misskey.niri.la
  • Misskey:2023.9.3-kinel.2

🛰 Backend (for server admin)

同上環境の開発環境でも確認しました。

  • Installation Method or Hosting Service:
  • Misskey: 13.x.x
  • Node: 20.x.x
  • PostgreSQL: 15.x.x
  • Redis: 7.x.x
  • OS and Architecture:
@anatawa12 anatawa12 added the ⚠️bug? This might be a bug label Oct 5, 2023
@anatawa12
Copy link
Member Author

Error: renderAnnounce: cannot render non-public note
    at ApRendererService.renderAnnounce (file:///Users/anatawa12/IdeaProjects/misskey/packages/backend/built/core/activitypub/ApRendererService.js:93:19)
    at NoteCreateService.renderNoteOrRenoteActivity (file:///Users/anatawa12/IdeaProjects/misskey/packages/backend/built/core/NoteCreateService.js:662:38)
    at file:///Users/anatawa12/IdeaProjects/misskey/packages/backend/built/core/NoteCreateService.js:546:53
    at NoteCreateService.postNoteCreated (file:///Users/anatawa12/IdeaProjects/misskey/packages/backend/built/core/NoteCreateService.js:572:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

@syuilo
Copy link
Member

syuilo commented Oct 5, 2023

そもそもダイレクト投稿がrenoteできるのが誤りでないかとも思っています

👍

@syuilo syuilo added 🐛Bug Unexpected behavior packages/backend Server side specific issue/PR and removed ⚠️bug? This might be a bug labels Oct 5, 2023
@anatawa12 anatawa12 changed the title ダイレクト投稿なRNを削除すると500エラーになる ダイレクト投稿をRNを削除すると500エラーになる Oct 5, 2023
@anatawa12 anatawa12 changed the title ダイレクト投稿をRNを削除すると500エラーになる ダイレクト投稿をRNできる Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants