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

特定の状況下で連合無しノートが連合される #12531

Closed
Kur0den opened this issue Dec 1, 2023 · 10 comments
Closed

特定の状況下で連合無しノートが連合される #12531

Kur0den opened this issue Dec 1, 2023 · 10 comments
Labels
⚠️bug? This might be a bug 🌌Federation The Federation/ActivityPub feature packages/backend Server side specific issue/PR

Comments

@Kur0den
Copy link

Kur0den commented Dec 1, 2023

💡 Summary

連合無しノートがピン止めしてある状態でそのアカウントを照会したことがない別インスタンスからそのユーザーを照会するとピン止めされている連合無しノートが連合されてしまう

一度連合されたノートは既に連合済みのインスタンスからでも閲覧/リアクション等ができる

🥰 Expected Behavior

連合無しノートは連合されない

🤬 Actual Behavior

連合無しノートが連合されてしまう

📝 Steps to Reproduce

  1. 連合なしのノートを作成
  2. 作成したノートをピン止め
  3. そのユーザーを照会したことがない別インスタンスからそのユーザーを照会する
  4. ピン止めされた連合無しのノートが見える

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL: koliosky.com, その他
* Misskey: 2023.11.1, 2023.12.0-beta1
    - 2023.11.1以前でも連合無しノートが連合されている状況を見たことがある

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:
@Kur0den Kur0den added the ⚠️bug? This might be a bug label Dec 1, 2023
@penginn-net
Copy link

連合済み(FF関係,リレー参加)のサーバーでも
リモートユーザの更新を行えば流れてきました

@Srgr0
Copy link
Collaborator

Srgr0 commented Dec 1, 2023

連合済み(FF関係,リレー参加)のサーバーでも リモートユーザの更新を行えば流れてきました

以下の環境では再現できませんでした。
送信:@srgr0@misskey.srgr0.com
受信:misskey.io、nijimiss.moe (それぞれ@srgr0でFF関係)
※送信側でノート投稿(連合なし)&ピン留め後、受信側でリモートユーザーの更新

@penginn-net
Copy link

penginn-net commented Dec 1, 2023

自分の場合
送信:@[email protected]
受信:@[email protected],@[email protected]
だと再現できるんですよね....

@1673beta
Copy link
Contributor

1673beta commented Dec 1, 2023

下記のアカウントで、送信側がピン留めしている状態でリモートユーザーの情報が更新されると再現しました。
送信: @[email protected]
受信: @[email protected]

@setaria430
Copy link
Contributor

これ不具合だったのですね。ピン留めノートはユーザ情報の中に入るので、リモートに送られるのも自然かと思ってました
チャンネルのノートを強引に連合させるのに便利で…

@Srgr0
Copy link
Collaborator

Srgr0 commented Dec 1, 2023

apiというかピン度目ノートはユーザーの情報にそのままrenderされるから連合したときに連合ありなし関係なく連合されてしまう
https://misskey.srgr0.com/notes/9mqb0fhmgw

「連合なし」のノートをピン留めするときに警告出して、ピン留めされたら「連合なし」状態を解除するくらいが良いのかも

@mei23
Copy link
Contributor

mei23 commented Dec 2, 2023

これ、フォロワー限定もダイレクトもAPから認証無しで内容まで見れるからぜいじゃくせいなきがするわ

@Srgr0
Copy link
Collaborator

Srgr0 commented Dec 6, 2023

fixed in #12541 ?

@mei23
Copy link
Contributor

mei23 commented Dec 6, 2023

fixed in #12541 ?

このIssueの「特定の状況下で連合無しノートが連合される」はなおってるわ。

自分以外見れなかったり連合先では見れない投稿が、普通に何も言われずにピン留め出来るのはどうなのってのはまだあるわ。

@KisaragiEffective
Copy link
Collaborator

fixed in #12541 ?

このIssueの「特定の状況下で連合無しノートが連合される」はなおってるわ。

自分以外見れなかったり連合先では見れない投稿が、普通に何も言われずにピン留め出来るのはどうなのってのはまだあるわ。

続きは #14616 で。表題のissueは解決したためこのissueは終了します。

@KisaragiEffective KisaragiEffective added packages/backend Server side specific issue/PR 🌌Federation The Federation/ActivityPub feature labels Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug 🌌Federation The Federation/ActivityPub feature packages/backend Server side specific issue/PR
Projects
Status: Done
Development

No branches or pull requests

7 participants