Skip to content

Merge 11.8.* / v5.0.be-6.9.* to 11.1? #2912

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

Closed
elarlang opened this issue Apr 7, 2025 · 11 comments
Closed

Merge 11.8.* / v5.0.be-6.9.* to 11.1? #2912

elarlang opened this issue Apr 7, 2025 · 11 comments
Assignees
Labels
1) Discussion ongoing Issue is opened and assigned but no clear proposal yet V11 (prev V6) _5.0 - rc1

Comments

@elarlang
Copy link
Collaborator

elarlang commented Apr 7, 2025

Spin off from #2908, Point 6.

# Description Level #v5.0.be
11.8.1 Verify that a cryptographic inventory is maintained and includes a documented transformation plan or mapping that outlines the migration path from current cryptographic algorithms and systems to those that use post-quantum cryptography. 3 v5.0.be-6.9.1
11.8.2 Verify that advancements in the field of post-quantum cryptography are being monitored in order to ensure that the application is aligned with emerging industry standards, and remains prepared for quantum threats. 3 v5.0.be-6.9.2

Those are clear inventory or documentation requirements (and not something you can check from the implementation) - by content those belong to "V11.1 Cryptographic Inventory and Documentation".

I also feel that 11.8.2 can be merged into 11.8.1.

@elarlang elarlang added 1) Discussion ongoing Issue is opened and assigned but no clear proposal yet _5.0 - rc1 V11 (prev V6) labels Apr 7, 2025
@jmanico
Copy link
Member

jmanico commented Apr 7, 2025

Agreed. Perhaps:

# Description Level #v5.0.be
11.8.1 Verify that a cryptographic inventory is maintained and includes a documented plan that outlines the migration path from current cryptographic solutions to those that use post-quantum cryptography. Additionally, verify that advancements in the field of post-quantum cryptography are being monitored to ensure alignment with emerging industry standards for quantum threats. 3 v5.0.be-6.9.1, v5.0.be-6.9.2

@elarlang
Copy link
Collaborator Author

elarlang commented Apr 8, 2025

ping @danielcuthbert @randomstuff

@randomstuff
Copy link
Contributor

randomstuff commented Apr 8, 2025

I like merging the PQC into 11.8.1.

I am still not certain that "advancements in the field of post-quantum cryptography are being monitored" is something we can verify. We can verify that the impact of PQC has been integrated in the documentation.

Can we simplify as:

Verify that a cryptographic inventory is maintained. This must include a documented plan that outlines the migration path to post-quantum cryptography in order to react to future quantum threats.

I would be tempted to generalize somewhat this formulation with something like:

Verify that a cryptographic inventory is maintained. This must include a documented plan that outlines the migration path to new cryptographic standards, such as post-quantum cryptography, in order to react to future threats.

@elarlang
Copy link
Collaborator Author

elarlang commented Apr 8, 2025

I like merging the PQC into 11.8.1.

11.8.1 is also about PQC...

Maybe 11.8.2 should be part of the section text?

@danielcuthbert
Copy link
Collaborator

Agree that validating monitoring is hard, but then again this is where ASVS could come in with the Appendix. I do actually like

Verify that a cryptographic inventory is maintained. This must include a documented plan that outlines the migration path to new cryptographic standards, such as post-quantum cryptography, in order to react to future threats.

that would be my recommendation

@elarlang
Copy link
Collaborator Author

As I understand, there is agreement, that the last quoted proposal covers current 11.8.1 and 11.8.2 and this requirement will be in section V11.1.

Current section text from 11.8 can be moved to 11.1 as well.

I'll PR that in later.


Agree that validating monitoring is hard, but then again this is where ASVS could come in with the Appendix

This is something that needs some further "translation" for me - is there a need for add something to AppendiX?

@jmanico
Copy link
Member

jmanico commented Apr 16, 2025

Agree that validating monitoring is hard, but then again this is where ASVS could come in with the Appendix. I do actually like

Verify that a cryptographic inventory is maintained. This must include a documented plan that outlines the migration path to new cryptographic standards, such as post-quantum cryptography, in order to react to future threats.

that would be my recommendation

+1 I think this is a great idea Daniel.

@elarlang
Copy link
Collaborator Author

elarlang commented Apr 16, 2025

PR #2912 (draft) - I did the requirement movement, chapter cleanup and related mapping updates.

Please add into the PR all needed and expected chapter text or AppendiX content if required.

@jmanico
Copy link
Member

jmanico commented Apr 16, 2025

Great PR. I just want to make a small wordsmith change for line 35. My grammar checker is just making a few suggestions. It keep suggesting we spell out ECC but not to bother with RSA which is a bit odd, but I like the rest. These are just small changes and not a big deal either way. If you are ok with this @elarlang I'll modify your PR.


The need to future-proof cryptographic systems against the eventual rise of quantum computing is critical. Post-Quantum Cryptography (PQC) refers to cryptographic algorithms designed to remain secure against attacks by quantum computers, which are expected to break widely used algorithms such as RSA and elliptic curve cryptography (ECC).

Please see the Appendix for current guidance on vetted PQC primitives and standards.

@elarlang
Copy link
Collaborator Author

If you are ok with this @elarlang I'll modify your PR.

Yes, go ahead, that's why it is draft.

Also, check Slack.

tghosth pushed a commit that referenced this issue Apr 17, 2025
* #2912 - reorg pqc inventory requirement

* Update 0x20-V11-Cryptography.md

minor wordsmithing

---------

Co-authored-by: Elar Lang <[email protected]>
Co-authored-by: Jim Manico <[email protected]>
@elarlang
Copy link
Collaborator Author

The changes that I proposed in the issue are now merged. If there is something needed from this for the appendix, please open a separate issue or PR to solve that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1) Discussion ongoing Issue is opened and assigned but no clear proposal yet V11 (prev V6) _5.0 - rc1
Projects
None yet
Development

No branches or pull requests

4 participants