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

Fix cade attack speed exploit #4204

Merged
merged 2 commits into from
Aug 22, 2023

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Aug 18, 2023

About the pull request

This PR fixes an exploit where dragging could be used to bypass an attack cooldown.

Explain why it's good for the game

Fixes #4184

Testing Photographs and Procedure

Screenshots & Videos

https://youtu.be/nIfXqQ6IBw0

Changelog

🆑 Drathek
fix: Fixed a cade attack exploit
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Aug 18, 2023
@harryob harryob added this pull request to the merge queue Aug 21, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 21, 2023
@morrowwolf morrowwolf added this pull request to the merge queue Aug 22, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Aug 22, 2023
@harryob harryob merged commit 05fe942 into cmss13-devs:master Aug 22, 2023
25 checks passed
cm13-github added a commit that referenced this pull request Aug 22, 2023
@Drulikar Drulikar deleted the Fix_Cade_Pull_Exploit branch August 22, 2023 20:06
github-merge-queue bot pushed a commit that referenced this pull request Sep 5, 2023
# About the pull request

This PR is a followup to #4204 that introduced a cooldown to grabbing to
fix an exploit. Unfortunately this broke logic for xeno grab intent
click though because that logic pre-emptively is adjusting next_move
(that xenos just override and ignore anyways), so now the cooldown check
is added to just ctrl+click. Technically this means that human
ctrl+click *could* be affected by this change, but I haven't currently
found any effects of this.

# Explain why it's good for the game

Fixes #4244 

# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

https://youtu.be/BMRnHIa967Q

</details>


# Changelog
:cl: Drathek
fix: Fixed xeno grab intent
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Xeno attack cade really fast bug
2 participants