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

/dev/app/newで作成したアプリではミュート関連の操作はできない? #2341

Closed
tamaina opened this issue Aug 19, 2018 · 10 comments · Fixed by #4611 or #4572
Closed
Labels
🧩API Issues regarding the interface between the server and the client. 🐛Bug Unexpected behavior

Comments

@tamaina
Copy link
Contributor

tamaina commented Aug 19, 2018

Summary

ソースを確認しただけで実装して確かめてみたわけではないが、エンドポイントのkindの値が、標準的なアカウント操作関連はaccount-read``account-writeなのに対しミュート関連はaccount/read``account/writeとなっている。
置換するコードは見つけられなかったが、仮にもしないとすれば/dev/app/newから作成したアプリはミュート関連の操作ができない(account/read``account/writeのpermissionは付与されないため)。

これは仕様ですか?

Environment


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@syuilo syuilo added 🐛Bug Unexpected behavior 🧩API Issues regarding the interface between the server and the client. labels Aug 19, 2018
@syuilo
Copy link
Member

syuilo commented Aug 19, 2018

ありがとうございます!ミスっぽいです

@tateisu
Copy link

tateisu commented Aug 19, 2018

'favorite-write' もないっぽいです

@tateisu
Copy link

tateisu commented Aug 20, 2018

パーミッション関連で問題を見つけた感じなので貼っておきます
https://misskey.xyz/notes/5b7aab7a403cad001f2bf161
https://misskey.xyz/notes/5b7aad63403cad001f2bf1e3

  • 既知のパーミッションの一覧をどこかに定義しておくべきです。
  • APIのコードで指定したパーミッションの文字列が既知のものかどうか、可能ならビルド時に分かるように工夫するべきです。でないとタイプミスを防止できません。
  • アプリ登録時に未知のパーミッションを許可しないようにするべきです。でないと後から新しいパーミッションを作る時に、アプリがこっそり指定していた文字列と競合する可能性が出てきます。

@tateisu
Copy link

tateisu commented Aug 23, 2018

https://github.com/syuilo/misskey/blob/master/src/server/api/endpoints/following/requests/list.ts
また新種のパーミッションを発見しました。
'following-read' となってますが、まず フォロリク申請を見れるのに適切な権限は 'following-write' であるべきだと思いますがどうでしょうか。

@tateisu
Copy link

tateisu commented Aug 24, 2018

kind: 'messaging-read'
kind: 'messaging-write',
kind: 'vote-write'

@tateisu
Copy link

tateisu commented Aug 30, 2018

お気に入り一覧の取得は favorites-read でした。 sの有無…

@tamaina
Copy link
Contributor Author

tamaina commented Mar 30, 2019

v11で直そう

@tamaina
Copy link
Contributor Author

tamaina commented Mar 30, 2019

'following-read' となってますが、まず フォロリク申請を見れるのに適切な権限は 'following-write' であるべきだと思いますがどうでしょうか。

そうでもなくない?

tamaina added a commit to tamaina/misskey that referenced this issue Mar 30, 2019
@tateisu
Copy link

tateisu commented Mar 30, 2019

フォローグラフはAPで公開されている情報なので認証なしでも読めますが、申請中のフォローリクエストはそうではありません。よりプライベートな情報です。

@tamaina
Copy link
Contributor Author

tamaina commented Mar 31, 2019

フォロー申請がプライベートなものであることは同意しますが、かといってフォロー申請を見ることに対するパーミッションをfollowing-writeにするのは分類としておかしいと思います。

フォロー申請の閲覧をfollowrequest-read、フォロー申請の承認/拒否操作をfollowrequest-writeとして分類しなおすべきだと思います。

syuilo pushed a commit that referenced this issue Apr 15, 2019
* Better permisson Fix #2341

* add kinds.ts

* test

* fix

* v11

* fix
nokotaro pushed a commit to nokotaro/misskey that referenced this issue Aug 15, 2021
Bumps [ip-cidr](https://github.com/ortexx/ip-cidr) from 2.1.2 to 2.1.3.
- [Release notes](https://github.com/ortexx/ip-cidr/releases)
- [Commits](https://github.com/ortexx/ip-cidr/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧩API Issues regarding the interface between the server and the client. 🐛Bug Unexpected behavior
Projects
None yet
3 participants