Merge pull request #224 from LasseBlaauwbroek/bootstrap-fix

Fix #223 by skipping the setting of object_id if it is empty
mirage · Feb 18, 2021 · d302574 · d302574
2 parents 8365bca + 50d4371
commit d302574
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/capnp-rpc-net/serialise.ml b/capnp-rpc-net/serialise.ml
@@ -70,7 +70,8 @@ module Make (EP : Capnp_core.ENDPOINT) = struct
       let b = Message.init_root ~message_size:100 () in
       let boot = Message.bootstrap_init b in
       Bootstrap.question_id_set boot (QuestionId.uint32 qid);
-      Schema.BuilderOps.write_string (Bootstrap.deprecated_object_id_get boot) object_id;
+      if not (String.equal object_id "") then
+        Schema.BuilderOps.write_string (Bootstrap.deprecated_object_id_get boot) object_id;
       Message.to_message b
     | `Call (qid, target, request, descs, results_to) ->
       let c = Msg.Request.writable request in