max-mapper / max-mapper/socket-sync

split websocket upgrade stuff out into its own module

Ouverte
#1 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
JavaScript
Étoiles
5
Forks
0
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

the websocket upgrade + session checking code needs to become its own module with a well defined API for checking a session and calling a callback with the upgraded socket

cc @nhq

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

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.

Piste de recherche

Commencez par localiser dans le dépôt le code de mise à niveau WebSocket et de vérification de session, puis suivez la manière dont les sockets mis à niveau sont actuellement gérés. Définissez la limite du module et l’API autour de la vérification de session ainsi que le callback du socket mis à niveau, puis vérifiez que le comportement de réplication existant reste intact.

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

Évaluation

Stack technique
javascript
Domaine
authentication, backend, networking
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

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