Null inside socket.data object's field
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 52/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- javascript
- Domaine
- frontend
Piste de recherche
Start by tracing registerVerboseListeners and createProxy, especially the path that processes each socket.data field during the connection event. Reproduce the case with a null field and verify that development mode no longer crashes while proxying other socket.data values.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I have middleware that sets socket.data and one of this object properties can be null. When socket.io-admin-ui in development mode it runs registerVerboseListeners function. Which laters calls createProxy function on "connection" event for every filed inside socket.data. Inside createProxy there is a check:
if (typeof obj !== "object") {
return obj;
}
but typeof null is also "object" so it will pass and will try to init new Proxy with null as target - new Proxy(null, ...) which will throw an error - "TypeError: Cannot create proxy with a non-object as target or handler" or similar.
In my case it also crash whole app when this error occur.
- Langage dominant
- Vue
- Étoiles
- 414
- Forks
- 109
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Aucun 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 socketio/socket.io-admin-ui
-
MaxListenersExceededWarningOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
socketio/socket.io-admin-ui#86 · 2 commentaires ·
-
Add support for DenoOuverteenhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
socketio/socket.io-admin-ui#84 · 2 commentaires · 2 réactions ·
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
socketio/socket.io-admin-ui#83 · 7 commentaires · 2 réactions ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
Toutes les issues de socketio/socket.io-admin-ui
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
openedx/frontend-app-authoring#3274 ·
Les mainteneurs répondent en général sous 1 jour
-
Unconfirmed bug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
luanti-org/luanti#17605 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
microsoft/fluentui-blazor#5364 ·
Les mainteneurs répondent en général sous 1 jour
-
COLEAD websiteOuverteneeds-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
wagtail/madewithwagtail#236 · 1 commentaire ·
Les mainteneurs répondent en général sous 4 jours
-
Design only Leadership Survey SLFS
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
bcgov/digital-journeys#2293 ·