Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add cmd-test arguments #324

Merged
ColtonWilley merged 3 commits intowolfSSL:masterfrom
aidangarske:fix-cmd-test
Dec 26, 2025
Merged

Add cmd-test arguments #324
ColtonWilley merged 3 commits intowolfSSL:masterfrom
aidangarske:fix-cmd-test

Conversation

@aidangarske
Copy link
Member

@aidangarske aidangarske commented Nov 4, 2025 •
edited by padelsbach
Loading

Description

  • Fixes errors here
  • Adds cmd line args for cmd test and ability to run select tests from do-cmd-test
  • Switch providers with use_wolf_provider and use_default_provider for unification rather than -provider args
  • Unify code to detect current setup with detect_wolfprovider_mode

@aidangarske aidangarske self-assigned this Nov 4, 2025
@aidangarske aidangarske marked this pull request as ready for review November 5, 2025 17:19
padelsbach
padelsbach reviewed Nov 5, 2025
View reviewed changes
padelsbach
padelsbach reviewed Nov 12, 2025
View reviewed changes
@padelsbach
Copy link
Contributor

padelsbach commented Nov 14, 2025

This branch on my fork has proposed changes to address my code review feedback: https://github.com/padelsbach/wolfProvider/tree/fix-cmd-test

@aidangarske
Copy link
Member Author

aidangarske commented Nov 14, 2025

This branch on my fork has proposed changes to address my code review feedback: https://github.com/padelsbach/wolfProvider/tree/fix-cmd-test

Can you push your fixed changes to this branch so we can get everything merged? We are going to be needing this to test replace default with the yocto replace default and the test here https://cloud.wolfssl-test.com/jenkins/job/wolfProvider/job/debian-packages/211/

@padelsbach padelsbach force-pushed the fix-cmd-test branch 10 times, most recently from 10ba2cb to 3754b1a Compare November 20, 2025 21:54
@sebastian-carpenter
Copy link
Contributor

sebastian-carpenter commented Dec 4, 2025 •
edited
Loading

In rsa-cmd-test.sh both sign_rsa_pss and verify_rsa_pss have -provider default as an argument to their sign/verify command.

Actually, it may be a bad idea to mess with that right now, so nevermind.

@padelsbach
Copy link
Contributor

padelsbach commented Dec 5, 2025

In rsa-cmd-test.sh both sign_rsa_pss and verify_rsa_pss have -provider default as an argument to their sign/verify command.

Great catch. We want to use use_wolf_provider and use_default_provider rather than the args. Fix pushed.

@padelsbach padelsbach force-pushed the fix-cmd-test branch 2 times, most recently from 76ec176 to d0d50db Compare December 6, 2025 19:05
@aidangarske aidangarske mentioned this pull request Dec 15, 2025
Open
1 task
@aidangarske
Copy link
Member Author

aidangarske commented Dec 23, 2025

Jenkins retest this please: Jenkins FIPS fix

ColtonWilley
ColtonWilley reviewed Dec 23, 2025
View reviewed changes
@padelsbach padelsbach force-pushed the fix-cmd-test branch 2 times, most recently from 17429f7 to c710cae Compare December 23, 2025 23:17
@ColtonWilley ColtonWilley merged commit 2f3af89 into wolfSSL:master Dec 26, 2025
370 checks passed
@aidangarske aidangarske mentioned this pull request Dec 30, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@padelsbach padelsbach padelsbach left review comments

@ColtonWilley ColtonWilley ColtonWilley approved these changes

Assignees

@ColtonWilley ColtonWilley

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@aidangarske @padelsbach @sebastian-carpenter @ColtonWilley