fix(network): a URL in SERVER_ADDRESS passes validation but builds an invalid ws URL
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 67/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- node.js, typescript
- Domaine
- networking
Piste de recherche
Start with modules/network/src/client/client-session.ts and modules/network/src/client/config.client.network.ts, then compare the accepted address formats with the network documentation at docs/docs/network/index.mdx. Check how the client constructs its WebSocket URL for a host, a URL with a scheme, and an IPv6 address. Done when accepted addresses produce valid WebSocket URLs and the documentation matches the supported formats.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Which package is this bug report for?
network
Issue description
The client config accepts ws / wss / http / https URLs and the docs say "IP address or URL", but the client then prefixes a scheme and appends a port: https://game.example.com becomes ws://https://game.example.com:4445. IPv6 addresses are not bracketed either. A setup the docs describe cannot connect.
Where:
modules/network/src/client/client-session.ts:45modules/network/src/client/config.client.network.ts:44docs/docs/network/index.mdx:84
Evidence:
@IsIpOrURL(undefined, undefined, { protocols: ["ws", "wss", "http", "https"] })
...
const url = `ws${wss ? "s" : ""}://${ip}:${port}`;
Suggested fix: Parse SERVER_ADDRESS with URL and take the scheme from it, or accept only bare hosts (in the validator and the docs). Bracket IPv6 addresses.
Found during a code audit of main at be72076.
Code sample
No response
Versions
- Engine
mainat be72076 - pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority
Medium (should be fixed soon)
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
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de NanoForge-dev/Engine
-
dependencies enhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
NanoForge-dev/Engine#513 ·
Les mainteneurs répondent en général sous 1 jour
-
bug chore tests
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
NanoForge-dev/Engine#511 ·
Les mainteneurs répondent en général sous 1 jour
-
docs(network): TSDoc says ICE_SERVERS defaults to an empty list, the code uses public STUN serversOuvertedocumentation good first issue modules:network
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
NanoForge-dev/Engine#509 ·
Les mainteneurs répondent en général sous 1 jour
-
documentation modules:ecs packages:asset packages:core
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
NanoForge-dev/Engine#508 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue modules:music modules:sound
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
NanoForge-dev/Engine#505 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de NanoForge-dev/Engine
Issues similaires
-
Mend: dependency security vulnerability
Difficulté 1/5 1-3 heures Accessibilité débutants 65/100
fdm-monster/fdm-monster#5563 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
be-BOP-io-SA/be-BOP#2849 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 79/100
Les mainteneurs répondent en général sous 1 jour
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
growilabs/growi#12246 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
🐛 The Rulekeeper - An index run that gives up on the lock ends SucceededPeut-être pris @github-actions l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
SSWConsulting/SSW.Rules#2809 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour