search

misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
9.95k stars 1.35k forks source link

アンテナの更新時にキーワードを与えないと内部エラーが報告される #14492

Closed KisaragiEffective closed 4 weeks ago

KisaragiEffective commented 1 month ago

💡 Summary

アンテナの更新時にキーワードを与えないと内部エラーが報告される

reported in https://misskey.niri.la/notes/9xqwlovlre

🥰 Expected Behavior

内部エラーではなく単なるエラーとして報告するべき。 現状の実装ではErrorが投げられているが、ApiErrorではないため内部エラーとしてフロントに出て不都合

🤬 Actual Behavior

内部エラーとして報告される

📝 Steps to Reproduce

  1. アンテナを作成する
  2. そのアンテナについて包含キーワード及び除外キーワードを空にして更新しようとする

💻 Frontend Environment

* Model and OS of the device(s): ?
* Browser: ?
* Server URL: misskey.niri.la
* Misskey: 2024.8.0-kinel.1

🛰 Backend Environment (for server admin)

n/a (not server admin)

Do you want to address this bug yourself?

Sayamame-beans commented 1 month ago

(そもそも: 多分フロント側でもvalidationされるべき)

samunohito commented 1 month ago

更新時だけでなく、作成時も同様の模様…

https://github.com/misskey-dev/misskey/blob/c1514ce91dc08481a092a789ee37da546cdb4583/packages/backend/src/server/api/endpoints/antennas/create.ts#L89-L91

samunohito commented 1 month ago

@KisaragiEffective 作成時も併せて対応いただくことは可能でしょうか…？

KisaragiEffective commented 1 month ago

@KisaragiEffective 作成時も併せて対応いただくことは可能でしょうか…？

やります！