Skip to content

enhance: 相互リンク機能の連合#704

Closed
kozakura913 wants to merge 5 commits into
MisskeyIO:iofrom
kozakura913:banner_federation
Closed

enhance: 相互リンク機能の連合#704
kozakura913 wants to merge 5 commits into
MisskeyIO:iofrom
kozakura913:banner_federation

Conversation

@kozakura913
Copy link
Copy Markdown

What

相互リンク機能で設定されたカテゴリ名、画像、リンク先、説明文などをユーザーのプロフィールに追加します
私の管理するforkから連合機能の部分をcherrypickしました
yojo-art/cherrypick#319

Why

独自拡張であっても、必要な場合に受け入れられるように情報を提供するのが最善だと考えます
最初は対応する実装は少ないかもしれませんが、その他の実装であっても必要であれば対応する事ができます

Additional info (optional)

#702 と関連

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

@sonarqubecloud
Copy link
Copy Markdown

@u1-liquid
Copy link
Copy Markdown
Member

PRありがとうございます。
頂いた変更について、以下の理由でマージを見送らせていただきます。

  1. カスタムフィールドでの実装は、同様のフィールドを実装しているMisskeyフォーク同士でのみ有効で、Fediverse全体での互換性に乏しいため。
  2. ActivityPub上定義されていないオブジェクトを生のJSONで転送しあう実装は、悪意のあるリモートサーバーからのXSS、Node.jsの脆弱性もしくは非常に大きいペイロードを受信した際の対策が困難であるため。
  3. リモートサーバーのバナーに対するモデレーションが困難であるため。

相互バナーリンク機能に関してMisskey.ioでは #702 の記載通り、この機能を搭載していない他のMisskeyサーバーやMastodon、PleromaなどFediverse上で一般的であるProfile metadataのattachmentフィールドとして、通常の追加情報のリンクの扱いで連合させる実装になる予定です。

@u1-liquid u1-liquid closed this Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants