Break the recursion in task result (#880)

Author: nvidianz

nvflare/private/fed/server/fed_server.py

@@ -510,7 +510,7 @@ def SubmitUpdate(self, request, context):
                         self.logger.info("ignored result submission since Server Engine isn't ready")
                         context.abort(grpc.StatusCode.OUT_OF_RANGE, "Server has stopped")
 
-                    shared_fl_context.set_prop(FLContextKey.SHAREABLE, shareable, private=False)
+                    shared_fl_context.set_prop(FLContextKey.SHAREABLE, shareable, private=True)
 
                     contribution_meta = contribution.client.meta
                     client_contrib_id = "{}_{}_{}".format(
@@ -593,7 +593,7 @@ def AuxCommunicate(self, request, context):
             fl_ctx.set_peer_context(shared_fl_context)
             shareable.set_peer_props(shared_fl_context.get_all_public_props())
 
-            shared_fl_context.set_prop(FLContextKey.SHAREABLE, shareable, private=False)
+            shared_fl_context.set_prop(FLContextKey.SHAREABLE, shareable, private=True)
 
             topic = shareable.get_header(ReservedHeaderKey.TOPIC)
 

tests/unit_test/app_common/widgets/in_time_model_selector_test.py

@@ -82,7 +82,7 @@ def test_model_selection(self, initial, received, expected):
                 },
             )
 
-            peer_ctx.set_prop(FLContextKey.SHAREABLE, dxo.to_shareable(), private=False)
+            peer_ctx.set_prop(FLContextKey.SHAREABLE, dxo.to_shareable(), private=True)
             fl_ctx = engine.fl_ctx_mgr.new_context()
             fl_ctx.set_prop(FLContextKey.PEER_CONTEXT, peer_ctx)