Skip to content

Added Post Quantum Server Testing Program#7

Merged
Haseeb702 merged 20 commits into
mainfrom
Initialize-pq-server-testing
Feb 9, 2026
Merged

Added Post Quantum Server Testing Program#7
Haseeb702 merged 20 commits into
mainfrom
Initialize-pq-server-testing

Conversation

@Haseeb702

Copy link
Copy Markdown
Collaborator

Purpose

  • To create a server side testing program to test signature generation time of post quantum and current algorithms

Changes

  • Added ML-DSA-65, Falcon-512, RSA-4096, ECDSA-384 signature generation code
  • Added a main file that runs 100 iterations of each algorithm with small and medium payloads
  • Added a README

@Haseeb702 Haseeb702 requested a review from alexcottner January 13, 2026 17:13
Comment thread post-quantum-server-testing/README.md
Comment thread post-quantum-server-testing/README Outdated
Comment thread post-quantum-server-testing/README.md
Comment thread post-quantum-server-testing/src/main.go Outdated
Comment thread post-quantum-server-testing/src/main.go Outdated
@Haseeb702 Haseeb702 force-pushed the Initialize-pq-server-testing branch from e0a5d9d to 5d12506 Compare January 14, 2026 20:27

@say-yawn say-yawn left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Minor suggestion and some questions:

  • why is there a GenerateKey() for both Falcon and RSA tests but not for MLDSA and ECDSA?
  • Why generate a new key for every test case for Falcon and RSA?
  • Why generate the RSA key locally rather than creating/storing it in the KMS?
  • Is the expectation that a key named mldsa and ecdsa with ML-DSA-65 and RSA-4096 respectively already created and stored in the KMS? If so, I recommend sharing the requirement/instructions about this on the README.

Comment on lines +14 to +16
sigName = "Falcon-512"
PublicKeySize = 897
PrivateKeySize = 1281

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

According to the internet (not official Go doc), you use uppercase if you are exporting the constant variable. Will you make changes to the variable names so they are consistent with the usage?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks. I'll change the variable names to uppercase.

@Haseeb702 Haseeb702 merged commit c68bb02 into main Feb 9, 2026
6 checks passed
@Haseeb702 Haseeb702 deleted the Initialize-pq-server-testing branch March 19, 2026 18:28
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.

3 participants