Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flappybird security #628

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Flappybird security #628

wants to merge 7 commits into from

Conversation

Riad-Attou
Copy link

Add security for score submission with key verification in flappybird

Previously, intercepting requests allowed users to modify scores.
Now, an additional security layer with a key mechanism ensures
scores cannot be modified with during transmission.

This commit goes hand-in-hand with the related changes in Titan.

Previously, intercepting requests allowed users to modify scores.
Now, an additional security layer with a key mechanism ensures
scores cannot be modified with during transmission.
@Daihecyy
Copy link
Contributor

Daihecyy commented Dec 5, 2024

See the section on linting and formatting to allow your code to pass : https://github.com/aeecleclair/Hyperion?tab=readme-ov-file#linting-and-formating
Remember to also edit the tests to ensure they still pass.
Since you implemented new behavior, it would be best if a new test reflected that.
Otherwise, great job !

@Daihecyy Daihecyy added enhancement New feature or request module labels Dec 5, 2024
Removed the 'key' column from the database models.

Improved the key verification logic.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants