Closed kakkokari-gtyih closed 1 week ago
failedだから関係ない気もする
Attention: Patch coverage is 0%
with 8 lines
in your changes missing coverage. Please review.
Project coverage is 19.42%. Comparing base (
ea2675e
) to head (dcdcc83
). Report is 6 commits behind head on develop.
Files with missing lines | Patch % | Lines |
---|---|---|
packages/frontend/src/components/MkSignin.vue | 0.00% | 8 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
failedだから関係ない気もする
ログインの次の指示をサーバーサイドから出すようになっていて、それは全部エラー扱いになっているのでこれであってる(/signin
というAPIリクエストを送るのにfailした場合のハンドリングという意味)
まあなんてこと
うーーーーーーーーむ
ログインがfailしたわけではないからとりあえず関数名変えたほうがいいかもしれない
ログインがfailしたわけではないからとりあえず関数名変えたほうがいいかもしれない
onLoginFailed
→ onSigninApiError
にした
APIレベルでエラーになってるの?
APIレベルでエラーになってるの?
ログインに必要な要素が足りない場合はわざと403を返すようにしてある (200はログイン成功な気がしたので)
うーーーーーーーーむ
設計がおかしい可能性がある
とりあえずマージ
🙏🏻
What
条件がごっそり抜けておった
Why
Fix https://github.com/misskey-dev/misskey/pull/14675#issuecomment-2393171003
Additional info (optional)
今のままでは特定のユーザーが締め出されてしまうので一旦マージするのも手
Checklist