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

Veryfymail.io を有効にしているとアカウント登録が極めて難しくなる #14926

Closed
1 task
fruitriin opened this issue Nov 9, 2024 · 2 comments
Labels
⚠️bug? This might be a bug

Comments

@fruitriin
Copy link
Contributor

💡 Summary

Veryfymail.io を有効にしていると、メールアドレスの確認のAPIがフォームのイベント全てに反応してリクエストされてしまい、APIのレートリミットに到達しやすい

🥰 Expected Behavior

@input.lazyなどlazy修飾子がついている、メールアドレスとしてvalidなフォーマットでなければ確認を行わない

🤬 Actual Behavior

全ての変化(1文字ずつ)に反応しているっぽい

📝 Steps to Reproduce

  1. メールアドレスの存在確認を有効にする
  2. アカウントを新規作成する

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey:

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@fruitriin fruitriin added the ⚠️bug? This might be a bug label Nov 9, 2024
@kakkokari-gtyih
Copy link
Contributor

全ての変化(1文字ずつ)に反応しているっぽい

debounceしているのでそうでもない

@fruitriin
Copy link
Contributor Author

改めて調査したところ、 verifymail.io側のレートリミットに引っ張られていそうなのでCloseしたいと思います

あと、一文字ずつチェックしてそうなのはユーザー名のほうで、emailのほうはちゃんとdebounceされてそうです
(どっちも /available という エンドポイントでその上のパスがトリミングされた結果、ネットワークタブで見間違えました。申し訳ない

@kakkokari-gtyih kakkokari-gtyih closed this as not planned Won't fix, can't repro, duplicate, stale Nov 30, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in [実験中] 管理用 Nov 30, 2024
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
Development

No branches or pull requests

2 participants