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

AP KeyがJSON-LDとして機能してない #13169

Closed
1 task done
mei23 opened this issue Feb 5, 2024 · 1 comment · Fixed by #13170
Closed
1 task done

AP KeyがJSON-LDとして機能してない #13169

mei23 opened this issue Feb 5, 2024 · 1 comment · Fixed by #13170
Labels
⚠️bug? This might be a bug

Comments

@mei23
Copy link
Contributor

mei23 commented Feb 5, 2024

💡 Summary

AP ActorのpublicKeyなどのtype: 'Key'@contextで関連付けられておらず、JSON-LDとして機能してない。

🥰 Expected Behavior

"type": "Key"
"owner": ...
"publicKeyPem": ...
:

↓ 正規化 (一番下)
image

🤬 Actual Behavior

image

📝 Steps to Reproduce

  1. Actor や Key (/users/:userId/publickey) のAP objectを取得する
  2. JSON-LD Playgroundなどで正規化

💻 Frontend Environment

N/A

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service: N/A
* Misskey: All including develop
* Node: N/A
* PostgreSQL: N/A
* Redis: N/A
* OS and Architecture: N/A

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@mei23 mei23 added the ⚠️bug? This might be a bug label Feb 5, 2024
@mei23
Copy link
Contributor Author

mei23 commented Feb 5, 2024

別リポジトリ

mei23#4800

詳細や実装

他の実装はどうなってる?
→ Mastodonとかには"type": "Key"は存在しない

なんでMisskeyにだけあるの?消しちゃえば?
→ 歴史的経緯でMisskey初期は (Actor#に添付するのではなく) ActorとKeyが別だった
→ AP /users/:userId/publickeyエンドポイントでオブジェクトを公開しているので消すわけにもいかなそう

その消せない AP /users/:userId/publickeyエンドポイントって必要なの?
→ 過去にそのエンドポイントをActorのkeyIdとして公開していたので、それで認識している実装があるかもしれない。

関連Issue

@contextをちゃんと付ける件 #7522

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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant