Skip to content
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

Test SSL ConnectionInfo certificate has the expected value #3401

Merged
merged 2 commits into from
Jan 22, 2025

Conversation

pepone
Copy link
Member

@pepone pepone commented Jan 22, 2025

Fix #2396

@externl
Copy link
Member

externl commented Jan 22, 2025

Seems to be a failure on Windows:

D:\a\ice\ice\cpp\test\Ice\info\AllTests.cpp:43: assertion `subjectName.find("O=ZeroC, Inc.") != string::npos || subjectName.find("O=ZeroC\\, Inc.") != string::npos' failed
-! 01/22/25 12:19:45.178 D:/a/ice/ice/cpp/test/Ice/info/msbuild/server/Win32/Release/server.exe: warning: connection exception:
   D:\a\ice\ice\cpp\src\Ice\StreamSocket.cpp:440 Ice::ConnectionLostException connection lost: The specified network name is no longer available.
   stack trace:
     0 Ice::SyscallException::SyscallException at D:\a\ice\ice\cpp\src\Ice\LocalExceptions.cpp:215
     1 Ice::SocketException::SocketException
     2 Ice::ConnectionLostException::ConnectionLostException at D:\a\ice\ice\cpp\src\Ice\LocalExceptions.cpp:312
     3 IceInternal::StreamSocket::finishRead at D:\a\ice\ice\cpp\src\Ice\StreamSocket.cpp:458
     4 Ice::SSL::Schannel::TransceiverI::finishRead at D:\a\ice\ice\cpp\src\Ice\SSL\SchannelTransceiverI.cpp:869
     5 Ice::ConnectionI::finishAsync at D:\a\ice\ice\cpp\src\Ice\ConnectionI.cpp:1132
     6 IceInternal::ThreadPool::startMessage at D:\a\ice\ice\cpp\src\Ice\ThreadPool.cpp:843
     7 Ice::ConnectionI::message at D:\a\ice\ice\cpp\src\Ice\ConnectionI.cpp:1175
     8 IceInternal::ThreadPool::run at D:\a\ice\ice\cpp\src\Ice\ThreadPool.cpp:722
     9 IceInternal::ThreadPool::EventHandlerThread::run at D:\a\ice\ice\cpp\src\Ice\ThreadPool.cpp:1078
    10 std::thread::_Invoke<std::tuple<void (__thiscall `anonymous namespace'::ServiceStatusManager::*)(void),A0xf54178e6::ServiceStatusManager *>,0,1> at C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\MSVC\14.42.34433\include\thread:61
    11 0000000074d96403
    12 00000000753f67f9
    13 0000000076f27f6d
    14 0000000076f27f3b
   
   local address = 127.0.0.1:14200
   remote address = 127.0.0.1:64291
!! 01/22/25 12:19:45.249 error: communicator not destroyed during global destruction.

@pepone pepone merged commit 7cf9671 into zeroc-ice:main Jan 22, 2025
23 of 24 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.

Review peer certificate tests
3 participants