Skip to content

Conversation

@toscalix
Copy link
Contributor

OID becomes a new property of the List

  • Property described in the crypto-algorithms-list-properties-description.md file
  • oid included in 33 algorithms

oid property included below id, as agreed This PR closes #20

@toscalix toscalix mentioned this pull request Oct 22, 2025
12 tasks
@toscalix
Copy link
Contributor Author

toscalix commented Oct 22, 2025

My original commit removed the license in every committed file. I added it back. I removed the empty line in every one of them

Copy link

@stevenc-stb stevenc-stb left a comment

Choose a reason for hiding this comment

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

oid review

toscalix and others added 7 commits October 29, 2025 14:55
I haven't found a correspondence between the oid of the PR and the algorithm. So I concur

Co-authored-by: stevenc-stb <[email protected]>
I agree

Co-authored-by: stevenc-stb <[email protected]>
I agree

Co-authored-by: stevenc-stb <[email protected]>
Obscene error from my side. Your review is pertinent

Co-authored-by: stevenc-stb <[email protected]>
Based on this seach, the right answer to this OID is not rc5 but rc5 symmetric cipher used with Cipher Block Chaining (CBC)  https://www.alvestrand.no/objectid/1.2.840.113549.3.9.html

Co-authored-by: stevenc-stb <[email protected]>
Co-authored-by: stevenc-stb <[email protected]>
Co-authored-by: stevenc-stb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New property: OID

2 participants