Skip to content

Rename wolfssh test certs - #831

Merged
JacobBarthelmeh merged 2 commits into
wolfSSL:masterfrom
gojimmypi:pr-rename-wolfssh-certs
Sep 3, 2025
Merged

JacobBarthelmeh merged 2 commits into
wolfSSL:masterfrom
gojimmypi:pr-rename-wolfssh-certs

Conversation

@gojimmypi

@gojimmypi gojimmypi commented Sep 3, 2025 •

Copy link
Copy Markdown
Contributor

Renames all local wolfSSH test certificates with _ssh suffix to avoid conflcts with those in wolfssl/test_certs.h

See also:

Testing:

wolfssl:

cd /mnt/c/workspace/wolfssl-$USER
make distclean
./autogen.sh
./configure --prefix=/mnt/c/workspace/wolfssh-$USER/wolfssl_install --enable-ssh
make
make install

wolfssh echoserver:

cd /mnt/c/workspace/wolfssh-$USER
./autogen.sh
./configure --with-wolfssl=/mnt/c/workspace/wolfssh-$USER/wolfssl_install
make

wolfssh client:

cd /mnt/c/workspace/wolfssh-$USER
./examples/client/client -u jill -h localhost -p 22222 -P upthehill
# or 
./examples/client/client -u jill -h 192.168.1.34 -p 22222 -P upthehill

@dgarske dgarske left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@gojimmypi

Copy link
Copy Markdown
Contributor Author

Good catch on the script. It appears to not be the script that generated the local wolfssh/certs_test.h.

I updated the script in 341a093 to match the current file contents.

@gojimmypi
gojimmypi requested a review from dgarske September 3, 2025 19:10
@JacobBarthelmeh
JacobBarthelmeh merged commit 4e087a0 into wolfSSL:master Sep 3, 2025
87 checks passed
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.

4 participants