Skip to content

Conversation

@gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented Sep 3, 2025

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

Copy link
Contributor

@dgarske dgarske left a comment

Choose a reason for hiding this comment

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

@gojimmypi
Copy link
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
93 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