fix(network): cap rooms, joiners and connections on the signaling server and the p2p host

Ouverte
  • Quelque chose ne va pas ?
#486 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
38/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
Active
Stack technique
typescript
Domaine
networking, security

Piste de recherche

Start with the cited room creation and joiner handling in modules/network/src/server/signaling.server.network.ts, then inspect joiner allocation in modules/network/src/server/peer-host.ts. Map the existing room and connection lifecycle before choosing limits and rate-limit behavior; the issue lists these goals but does not specify threshold values or validation rules. Done means enforcing agreed limits and validating bounded offers, with tests covering the listed abuse cases.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug modules:network
Which package is this bug report for?

network

Issue description

Every host connection creates a room and every joiner is added, with no cap on rooms, joiners per room, connections per IP or frame rate. The 404-versus-upgrade answer tells whether a room exists and nothing throttles it (code space 32^6). On the p2p host, each new signal id that sends an offer allocates an RTCPeerConnection, with no limit on _joiners and no validation of offer / candidate.

Where:

Evidence:

if (room !== null && !this._rooms.has(room)) { return new Response("Unknown room", { status: 404 }); }
this._rooms.set(room, { host: webSocket, joiners: new Map() });
...
if (!joiner) { if (message.type !== "offer") return; joiner = this.addJoiner(signalId); }

Suggested fix: Cap rooms, joiners per room, connections per IP and joiners not yet welcomed. Rate-limit failed room lookups and joiner frames. Check the offer shape and bound the sdp length.

Found during a code audit of main at be72076.

Code sample

No response

Versions
  • Engine main at be72076
  • pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority

High (immediate attention needed)

I have tested this issue on a development release

be72076

Langage dominant
TypeScript
Étoiles
7
Forks
2
Merge moyen
1 j 17 h
PR mergées (30 j)
17

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de NanoForge-dev/Engine

Toutes les issues de NanoForge-dev/Engine

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.