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

fix(frontend): ユーザー登録完了時にサインインAPIを別途使用していたのを修正 #14738

Closed kakkokari-gtyih closed 3 days ago

kakkokari-gtyih commented 3 days ago

What

signup apiで元から提供されているtokenを利用してログインするように

Why

Fix #14736

Additional info (optional)

Checklist

codecov[bot] commented 3 days ago

Codecov Report

Attention: Patch coverage is 0% with 68 lines in your changes missing coverage. Please review.

Project coverage is 41.37%. Comparing base (4a356f1) to head (346f09a). Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
...es/frontend/src/components/MkSignupDialog.form.vue 0.00% 47 Missing :warning:
packages/frontend/src/pages/settings/accounts.vue 0.00% 9 Missing :warning:
packages/frontend/src/account.ts 0.00% 4 Missing :warning:
...ackages/frontend/src/components/MkSigninDialog.vue 0.00% 3 Missing :warning:
packages/frontend/src/components/MkSignin.vue 0.00% 2 Missing :warning:
...ackages/frontend/src/components/MkSignupDialog.vue 0.00% 2 Missing :warning:
...ackages/frontend/src/components/MkUserCardMini.vue 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #14738 +/- ## =========================================== + Coverage 39.58% 41.37% +1.79% =========================================== Files 1553 1557 +4 Lines 195033 200808 +5775 Branches 3624 3677 +53 =========================================== + Hits 77200 83094 +5894 + Misses 117230 117146 -84 + Partials 603 568 -35 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

kakkokari-gtyih commented 3 days ago

ready

syuilo commented 3 days ago

これcypressでエラーにならなかったのって単にemail無し+CAPTCHA有りパターンのテストが無かったからってことかしら

kakkokari-gtyih commented 3 days ago

これcypressでエラーにならなかったのって単にemail無し+CAPTCHA有りパターンのテストが無かったからってことかしら

たぶんそう

syuilo commented 3 days ago

これcypressでエラーにならなかったのって単にemail無し+CAPTCHA有りパターンのテストが無かったからってことかしら

そもそもcypressではCAPTHCAは無効だったわ

syuilo commented 3 days ago

🙏🏻

kleuzjatob commented 3 days ago

対応された後に質問を失礼いたします。

サインインAPIを別途使用していたのを修正

先日発生したらしいスパム騒動の関連か不明ですが、10日ほど前から弊サーバーにて無作為な username の登録が相次いで発生しており、ブロック対応のため調査したところ、signup 後に走らせているらしい signin(最新版は signin-flow ?)を故意に止めて Misskey 側にログイン情報を記録させないよう図っている事が伺えました。スパム対策としてチケットを発行しようとしておりましたが、本件にて序でに解消された形でしょうか?