-
Notifications
You must be signed in to change notification settings - Fork 14
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
Smoketest fails with crypto error #644
Comments
this seem like the bug fixed in #636. make sure we use fixed tunneler version in smoke test |
I don't recall which version of zet was in use when this occurred. Based on commits to smoketest.go it could have been either zet 0.22.25 (csdk 0.36.7, which did not contain #636) or zet 0.22.28 (csdk 0.36.10, which did contain #636). @plorenz do you remember the zet version that was in play? |
testing file attachment |
Thought I had attached the logs, but I hadn't. From the logs ^^ |
Hmm. The only zet version that used csdk 0.36.9 was zet 0.22.26 (and .27, which was only pre-release). Perhaps the smoketest was using a custom kit when this issue came up? In any case #636 debuted in ziti-sdk-c 0.36.9, so whatever happened during this smoke test was not fixed by #636. |
Logs attached.
Example run:
Client side shows crypto error:
hosting side doesn't show error:
The text was updated successfully, but these errors were encountered: