Skip to content

Updated the crytpo page, added relevant links and more up to date information #349

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

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

pUrGe12
Copy link
Collaborator

@pUrGe12 pUrGe12 commented Feb 3, 2025

Summary :

This PR is for the issue #346. It is to update the principles of cryptography section with more upto date examples and relevant links. Keeping in mind the contribution guidelines and running them locally first, I made the following changes:

  1. Stream cipher examples, RC4 to ChaCha20 (because the RC4 algorithm has many vulnerabilities like biases in the key scheduling algorithm)
  2. Updated with note on DES not being a standard, and added AEAD schemes.
  3. Corrected spelling errors and other grammatical issues
  4. Updated the wordlists to handle acronyms for these new crypto-systems.

Ran and tested according to the guidelines locally.

Description for the changelog :

Added acronyms to wordlists and updated the crypto page

Other info :

closes #346

@jgadsden jgadsden self-requested a review February 3, 2025 13:39
@jgadsden jgadsden added the content suggest content label Feb 3, 2025
@jgadsden
Copy link
Collaborator

jgadsden commented Feb 3, 2025

thanks @pUrGe12 , and the automated tests do not run because you are requesting commit into revise-crypto from pUrGe12:revise-crypto, rather than into main

@pUrGe12
Copy link
Collaborator Author

pUrGe12 commented Feb 4, 2025

Do you need me to create for the main branch?

Copy link
Collaborator

@jgadsden jgadsden left a comment

Choose a reason for hiding this comment

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

good to have these changes in @pUrGe12 , very useful to keep the crypto page up to date
the link check fail is not to do with this change, so it can be merged into branch revise-crypto

@jgadsden jgadsden merged commit c6185af into OWASP:revise-crypto Feb 4, 2025
@pUrGe12 pUrGe12 deleted the revise-crypto branch February 10, 2025 17:20
@jgadsden jgadsden linked an issue Feb 14, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content suggest content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Revise section: Principles of cryptography
2 participants