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

Make rate limit for migrations less strict #15153

Open
1 task
sugar700 opened this issue Dec 19, 2024 · 0 comments
Open
1 task

Make rate limit for migrations less strict #15153

sugar700 opened this issue Dec 19, 2024 · 0 comments
Labels
✨Feature This adds/improves/enhances a feature

Comments

@sugar700
Copy link

sugar700 commented Dec 19, 2024

Summary

Change i/move endpoint rate limit to something like 10 users per hour.

Purpose

Currently i/move endpoint has a rate limit configured to 5 per day.

This is problematic, as there are various reasons why migrations could fail, and if the button is clicked a lot despite not fixing issues, it's necessary to wait an entire day afterwards. I saw a lot of cases of people hitting this rate limit due to not understanding the process well.

I don't think making rate limit less strict is a problem, as overall the migration process is pretty cheap if it fails, doing one remote look-up at worst.

It gets a lot more expensive if all the early checks passed, however this mostly doesn't matter, as the account cannot migrate once already migrated.

Do you want to implement this feature yourself?

  • Yes, I will implement this by myself and send a pull request
@sugar700 sugar700 added the ✨Feature This adds/improves/enhances a feature label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature
Projects
Development

No branches or pull requests

1 participant