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

Rolling xor #452

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Rolling xor #452

wants to merge 4 commits into from

Conversation

masq
Copy link
Contributor

@masq masq commented Dec 27, 2018

Hey, first time contributing to CyberChef, thought I'd just try to implement an extra option to an existing operation. Tried to take a swing at #426. I added a few "Rolling" options, not sure entirely if this accomplishes exactly the requested operation, but it should be a good start.

Please let me know how it is and what else I can do!

@CLAassistant
Copy link

CLAassistant commented Dec 27, 2018

CLA assistant check
All committers have signed the CLA.

BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this pull request May 29, 2022
[FIX] General issues and improvements
Fixes gchq#452
Fixes gchq#454
Fixes gchq#455
Fixes gchq#463
Fixes gchq#479
Fixes gchq#482
Fixes gchq#483
Fixes gchq#485
Fixes gchq#486
Fixes gchq#487
@n1474335
Copy link
Member

Hi @masq, thanks very much for this PR. I'd rather not add more methods to inbuilt prototypes like Array and String as it can effect performance. Could you modify this so that the functions are standalone and then I will merge it. Thanks!

@n1474335 n1474335 added the tidying required Code improvements needed before merge label Mar 24, 2023
@masq masq closed this Mar 31, 2023
@masq masq reopened this Mar 31, 2023
],
},
{ op: "To Binary", args: ["Space"] },
],
},
Copy link
Collaborator

@mattnotmitt mattnotmitt Feb 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs some tests for Rolling... schemes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement tidying required Code improvements needed before merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants