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.

Sign up for GitHub

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

Jump to bottom

The API is throtlling the openverse.org #3501

Closed
krysal opened this issue Dec 8, 2023 · 0 comments · Fixed by #3486
Closed

The API is throtlling the openverse.org #3501

krysal opened this issue Dec 8, 2023 · 0 comments · Fixed by #3486
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents 🧱 stack: api Related to the Django API 🧱 stack: frontend Related to the Nuxt frontend

Comments

@krysal
Copy link
Member

krysal commented Dec 8, 2023

Description

The following error is still getting events today (4.8k in the last 30 days) in Sentry.

AxiosError: Request failed with status code 429
  at settle (../node_modules/.pnpm/[email protected]/node_modules/axios/lib/core/settle.js:19:12)
  at onloadend (../node_modules/.pnpm/[email protected]/node_modules/axios/lib/adapters/xhr.js:111:7)

One of the URLs is https://openverse.org/search/?license_type=commercial,modification&q=football but it is happening with others as well.

Additional context

Sentry info.
@krysal krysal added 🟧 priority: high Stalls work on the project or its dependents 🛠 goal: fix Bug fix 💻 aspect: code Concerns the software code in the repository 🧱 stack: api Related to the Django API 🧱 stack: frontend Related to the Nuxt frontend labels Dec 8, 2023
@openverse-bot openverse-bot moved this to 📋 Backlog in Openverse Backlog Dec 8, 2023
@krysal krysal changed the title The API is throtlling the Frontend The API is throtlling the openverse.org Dec 8, 2023
@AetherUnbound AetherUnbound moved this from 📋 Backlog to 📅 To Do in Openverse Backlog Dec 11, 2023
@AetherUnbound AetherUnbound linked a pull request Dec 11, 2023 that will close this issue
Add referrer based throttle scope #3486
Merged
7 tasks
@AetherUnbound AetherUnbound moved this from 📅 To Do to 🏗 In Progress in Openverse Backlog Dec 11, 2023
@openverse-bot openverse-bot moved this from 🏗 In Progress to ✅ Done in Openverse Backlog Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents 🧱 stack: api Related to the Django API 🧱 stack: frontend Related to the Nuxt frontend
Projects
Openverse Backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add referrer based throttle scope WordPress/openverse
1 participant
@krysal