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

addition of v6 and PQC #2124

Closed
wants to merge 18 commits into from
Closed

addition of v6 and PQC #2124

wants to merge 18 commits into from

Conversation

falko-strenzke
Copy link
Contributor

This pull request is a mirror of pqc-thunderbird/rnp#35

TJ-91 and others added 18 commits September 6, 2023 15:42
correction of key and signature related code based on rnp feedback

address some more points of PR reiew

address some more points of PR reiew

rename parse_v3 and update comments

correction of key version handling according to rnp feedback

changes to v6 key & sig

remove redundant ifdef
addressed various feedback comments
…/eddsa abstraction for the following PQC integration
correction of dilithium based on rnp feedback
correction of kyber based on rnp feedback
conditionally compile RNP_ALGNAME_* macros

address some more points of PR review (JRH)

corrections of key-gen ui according to rnp feedback

SHA3 hash binding for dilithium-exdsa-composite
manual correction of code formatting that still deviates
update copyright (2)
pqc draft: remove packet hash

pqc draft: add ecc pubkey to hashed ecdh key share

pqc draft: check AES for v3 PKESK
check SPHINCS+ Botan feature
fix Botan deprecation in kyber code
fix build system
remove comments
reverse ifdefs and config.h inclusion in rnp.h
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Patch coverage: 74.83% and project coverage change: -0.10% ⚠️

Comparison is base (22f5b19) 83.89% compared to head (6332695) 83.79%.

❗ Current head 6332695 differs from pull request most recent head f953a39. Consider uploading reports for the commit f953a39 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2124      +/-   ##
==========================================
- Coverage   83.89%   83.79%   -0.10%     
==========================================
  Files         161      161              
  Lines       32249    32404     +155     
==========================================
+ Hits        27054    27154     +100     
- Misses       5195     5250      +55     
Files Changed Coverage Δ
src/lib/crypto.cpp 72.16% <ø> (ø)
src/lib/crypto/ec.cpp 76.92% <ø> (ø)
src/lib/crypto/ecdh.cpp 80.57% <ø> (ø)
src/lib/crypto/ecdsa.cpp 71.94% <ø> (ø)
src/lib/crypto/hash.hpp 100.00% <ø> (ø)
src/lib/crypto/hash_common.cpp 94.91% <0.00%> (-3.34%) ⬇️
src/lib/crypto/rng.cpp 77.77% <ø> (ø)
src/lib/ffi-priv-types.h 100.00% <ø> (ø)
src/lib/pgp-key.h 91.66% <ø> (ø)
src/lib/types.h 100.00% <ø> (ø)
... and 33 more

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@falko-strenzke
Copy link
Contributor Author

There are at least two problems:

  • renaming the repositoy
  • false positive by CodeQL

Thus I close this PR again and will create it anew.

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.

2 participants