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

api-docにapi/emojisとapi/emojiがそれぞれ2つずつある #12640

Closed
1673beta opened this issue Dec 13, 2023 · 7 comments
Closed

api-docにapi/emojisとapi/emojiがそれぞれ2つずつある #12640

1673beta opened this issue Dec 13, 2023 · 7 comments
Labels
⚠️bug? This might be a bug

Comments

@1673beta
Copy link
Contributor

💡 Summary

api-doc中にemojisとemojiがPOSTとGETの2つずつある。
検索をかけるとなぜかどっちもGETメソッドとして表示される。
画像はkoliosky.comでemojiで検索をかけたもの。
image

🥰 Expected Behavior

1つずつ(のはず)

🤬 Actual Behavior

2つ以上ずつある

📝 Steps to Reproduce

  1. api-docを開く
  2. なんか増えてる
  3. emojiあるいはemojisで検索をかけると画像のようになる

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL: koliosky.com, misskey.systems
* Misskey: 2023.12.0-beta.3

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service: systemd
* Misskey: 2023.12.0-beta.3
* Node: 20.5.1
* PostgreSQL: 16
* Redis: 7.2.3
* OS and Architecture: Ubuntu 22.0.4 LTS
@1673beta 1673beta added the ⚠️bug? This might be a bug label Dec 13, 2023
@1673beta 1673beta changed the title api-docにapi/emojisとapi/emojiが2つある api-docにapi/emojisとapi/emojiがそれぞれ2つずつある Dec 13, 2023
@kakkokari-gtyih
Copy link
Contributor

api.jsonではちゃんと1つずつ出ているのでビューワー側の問題かも

@Lumisilk
Copy link

Lumisilk commented Jan 6, 2024

別の質問というか、例えば /emojis は POST と GET 両方存在しているのはなぜでしょうか… 返り値が同じなのに。
この問題で両者の OpenAPI の operation ID が被っていて、OpenAPI 3.1 の仕様に反しているはず。

それとも今の Misskey API は OpenAPI 使わず、他に新設のなにかを使う方向ですか…?

(バックエンドにあまり詳しくなくて何か間違えたらすみません)

@anatawa12
Copy link
Member

GETとPOST両方あるのは

  • MisskeyのAPIは原則POST
  • 例外的に一部APIはブラウザ等キャッシュを効かせるためGETも受け付ける

という仕様のはずです

@Sayamame-beans
Copy link
Member

triage: 現状仕様なのでclose…?

@samunohito
Copy link
Member

この問題で両者の OpenAPI の operation ID が被っていて、OpenAPI 3.1 の仕様に反しているはず。

これは直すべきだと考えているので、openのままでお願いします 🙏
(手が回ってない)

@samunohito
Copy link
Member

ああ、でも… #13498 に包括できるかも…?

@zyoshoka
Copy link
Contributor

少なくとも本 issue の本題である

api-doc中にemojisとemojiがPOSTとGETの2つずつある。
検索をかけるとなぜかどっちもGETメソッドとして表示される。

については #14152 にて直っているようですので close します。OpenAPI Specification に沿っていない問題に関しては #13498 にまとめましょう

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

No branches or pull requests

7 participants