Skip to content

Input checking #2

Description

@Meuritz

The game dosen't check the followings:

  • When playing a card from the hand, if the cards are less than 9, an invalid index will be sent to the server, and the game will crash
  • If the client sends a string or an invalid input to the server (varies whit the move chosen) the server will crash
  • when getting a card from the board, a player can take an "empty card", if the player takes 2 times the same card, that card will be deleted
  • possible error yet to be tested, if a player tries to take a card from another one that has no cards

Possible fixes

  • send each time the number of available cards in the hand, and check the input for gibberish
  • trow an error if a player tries to play an empty card, and recover by making him chose another one
  • trow an error when a player tries to take a card from a player whit no cards in his hands, and make him do another choice or dont display the player whit 0 cards at all

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't workinghelp wantedExtra attention is needed

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions