fix: install signal handlers should occur after plugins started

pact-foundation · Jul 3, 2024 · abd1815 · abd1815
1 parent c80e188
commit abd1815
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/internal/native/message_server.go b/internal/native/message_server.go
@@ -330,8 +330,8 @@ func (m *MessageServer) UsingPlugin(pluginName string, pluginVersion string) err
 	cPluginVersion := C.CString(pluginVersion)
 	defer free(cPluginVersion)
 
-	InstallSignalHandlers()
 	r := C.pactffi_using_plugin(m.messagePact.handle, cPluginName, cPluginVersion)
+	InstallSignalHandlers()
 
 	// 1 - A general panic was caught.
 	// 2 - Failed to load the plugin.

diff --git a/internal/native/mock_server.go b/internal/native/mock_server.go
@@ -555,8 +555,8 @@ func (m *MockServer) UsingPlugin(pluginName string, pluginVersion string) error
 	cPluginVersion := C.CString(pluginVersion)
 	defer free(cPluginVersion)
 
-	InstallSignalHandlers()
 	r := C.pactffi_using_plugin(m.pact.handle, cPluginName, cPluginVersion)
+	InstallSignalHandlers()
 
 	// 1 - A general panic was caught.
 	// 2 - Failed to load the plugin.