fix: avoid empty installation id

langgenius · Nov 28, 2024 · 72d842f · 72d842f
1 parent 2c659c7
commit 72d842f
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/internal/core/plugin_manager/remote_manager/hooks.go b/internal/core/plugin_manager/remote_manager/hooks.go
@@ -118,8 +118,10 @@ func (s *DifyServer) OnClose(c gnet.Conn, err error) (action gnet.Action) {
 	// uninstall plugin
 	if plugin.assetsTransferred {
 		if _mode != _PLUGIN_RUNTIME_MODE_CI {
-			if err := plugin.Unregister(); err != nil {
-				log.Error("unregister plugin failed, error: %v", err)
+			if plugin.installationId != "" {
+				if err := plugin.Unregister(); err != nil {
+					log.Error("unregister plugin failed, error: %v", err)
+				}
 			}
 
 			// decrease current connection