Respect game visibility
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Commencez par GameService#markDirty, DelayedResponse, ClientService#broadcastDelayedResponses et GameResponsesTransformer#game, puis examinez les tests unitaires associés. C’est terminé lorsque les parties privées ne sont pas diffusées, que les parties réservées aux amis ne sont envoyées qu’aux amis, que la visibilité est omise des réponses de jeu du client, que GameAccess est supprimé et que les tests modifiés passent.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Currently, the server broadcasts all games to all players. However, games marked as PRIVATE should not be broadcasted at all and games marked as FRIENDS should only be sent to friends.
- Remove the unused
com.faforever.server.game.GameAccess - Don't serialize
visibilitywhen sending games to client (incom.faforever.server.integration.legacy.transformer.GameResponsesTransformer#game) - Implement a method similar to
com.faforever.server.game.GameService#markDirtythat allows specifying a list of players to send to - Extend
com.faforever.server.client.DelayedResponseso that a list of receivers can be specified. The list should be nullable (so make its getter return an Optional), in which case it's considered a broadcast (document this in the getter) - Refactor
com.faforever.server.client.ClientService#broadcastDelayedResponsesso that it either broadcasts the response or sends it to the list of recipients. - Adjust and add unit tests accordingly
- Langage dominant
- Java
- Étoiles
- 12
- Forks
- 5
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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 FAForever/faf-java-server
-
Server closes peers to ppl disconnecting from serverPeut-être à nouveau libre @1-alex98 l’a pris il y a 2844 jours, et aucune pull request n’est ouverte. Ouvertebug
FAForever/faf-java-server#112 · 1 personne assignée ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
FAForever/faf-java-server#110 · 1 commentaire ·
-
Return correct error code if player is unavailable for matchmaking because they're in a gameOuvertebeginner-friendly
Difficulté 3/5 1-2 jours Accessibilité débutants 20/100
FAForever/faf-java-server#100 ·
-
Measure ladder1v1 search timeOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
FAForever/faf-java-server#99 ·
-
New Ban Revoke StructurePeut-être à nouveau libre @1-alex98 l’a pris il y a 2923 jours, et aucune pull request n’est ouverte. Ouverteenhancement
FAForever/faf-java-server#95 · 1 personne assignée ·
Toutes les issues de FAForever/faf-java-server
Issues similaires
-
Difficulté 2/5 Moins d'une heure Accessibilité débutants 70/100
Vakore/ZappierGames#248 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
thelastpickle/cassandra-reaper#1748 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 3 jours
-
A proxy that fails to start logs a RejectedExecutionException stack trace from its shutdown hookOuvertefollow-up
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Sanne/incus-spawn#1953 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
OneBusAway/maglev#1917 · 1 commentaire ·
Les mainteneurs répondent en général sous 3 jours