Fix erroneous JSON data loading #5
No reviewers
Labels
No Label
security
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Blocks
#4 Running '!add games' deletes all state data upon restarting twice
LibreGaming/matchbot
Reference: LibreGaming/matchbot#5
Loading…
Reference in New Issue
No description provided.
Delete Branch "seabass/matchbot:fix-json-loading"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This pull request resolves issue #4, fixing a security vulnerability described in that issue.
Commits
Move processing out of clojure.data.json/read call
This commit removes from the call to clojure.data.json/read the conversion of the serialised list of players into a Clojure set, and adds an independent function to do this task instead.
Since the structure of the serialised data is known in advance, the recursive characteristics of clojure.data.json are not needed.