Chocobozzz/PeerTube

Support disabling rates limit

Open

#6 089 ouverte le 6 déc. 2023

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)TypeScript (11 911 stars) (1 362 forks)batch import
Type: Feature Request :sparkles:good first issue

Description

Describe the problem

This is my current rate limits configure

rates_limit:
  api:
    window: 10 seconds
    max: 0
  login:
    window: 5 minutes
    max: 0
  signup:
    window: 5 minutes
    max: 0
  ask_send_email:
    window: 5 minutes
    max: 0
  receive_client_log:
    window: 10 minutes
    max: 0
  plugins:
    window: 10 seconds
    max: 0
  well_known:
    window: 10 seconds
    max: 0
  feeds:
    window: 10 seconds
    max: 0
  activity_pub:
    window: 10 seconds
    max: 0
  client:
    window: 10 seconds
    max: 0

When I visit the page, I received 429 Too many request error.

If I want to set all peertube rate limits to unlimit in v6.0.1, what value should I set?

Additional information

  • PeerTube instance URL:
  • PeerTube version: v6.0.1
  • NodeJS version: v18.19.0
  • Ffmpeg version:
  • Redis version: 7.0
  • PostgreSQL version: 13.3

Guide contributeur

Stack technique
typescriptnodejsredispostgresql
Domaine
backend
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
needs maintainer response
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
needs investigation
Prérequis
basic YAML understandingPeerTube admin experience
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Investigate the rate limiting configuration parsing and middleware implementation in PeerTube. The issue suggests that setting max to 0 should disable rate limiting but results in a 429 error. Determine if the code correctly handles max=0 as unlimited or if there is a bug. Check the documentation for rate limiting and propose either a fix to allow max=0 to disable or a clearer configuration option like 'enabled: false'. Look at files related to rate limiting middleware and configuration validation.