acmutd / acmutd/meteormate-client

feat: Rate limting on backed

Abierto
#123 0 comentarios 0 reacciones 1 asignado Reclamado por @PyDev19 Ver en GitHub
Type: Feature Request
Lenguaje dominante
TypeScript
Estrellas
0
Forks
0
Merge medio
1 d 3 h
PR fusionados (30 d)
2

Descripción

### Problem / Motivation

A single, very motivated, UTD student can use inspect element to find the API endpoints and their bearer token. They can then use this info to DDOS our endpoints. Vercel somewhat stops this with their serverless system, but if we ever move to lambdas or non-serverless, it will be a bigger issue. So we need rate limitation baked into the backend as well as basic rate limiting to the frontend.

### Proposal

We can easily add this via `fastapi-limiter` for a simple rate limiting per IP address but for more complex production ready cases we need to implement a redis cahce for his to work and do it per UID instead of IP.

### Acceptance criteria

_No response_

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.