bug fixed query_proc

zhangmeng

2023-01-29 e4cb33f25012c534f0582d485c4f491f9dd3cd25

 src/interface_bus_api.cpp

@@ -31,6 +31,7 @@
        bus *b = new bus;
        bhome_msg::ProcInfo pi;
        if (pi.ParseFromArray(proc_info, proc_info_len)) get<8>(*b) = pi.proc_id();
        std::move(pi);

        const auto& url_hb = get_url(URLHeartBeat);
        respond_survey(url_hb,string{(char*)proc_info,(size_t)proc_info_len},&get<5>(*b));
@@ -106,8 +107,11 @@
    for(int i = 0; i < mtl.topic_list_size(); i++){
        mtl2.add_topic_list(mtl.topic_list(i));
    }
    std::move(mtl);

    string msg(mtl2.SerializeAsString());
    std::move(mtl2);

    const auto& url = get_url(URLRegTopic);
    if (url.empty()) {
        set_last_error("bus_register_topics url empty");
@@ -135,7 +139,7 @@
        return false;
    }

    auto url(get_url(URLQueryTopic));
    const auto& url = get_url(URLQueryTopic);
    if (url.empty()) {
        set_last_error("bus_query_topic_address url empty");
        return false;
@@ -168,12 +172,12 @@
        PRNTVITAG("handle is null");
        return false;
    }
    if (!query || query_len <= 0) {
        PRNTVITAG("query is null");
        return false;
    }
    // if (!query || query_len <= 0) {
    //     PRNTVITAG("query is null");
    //     return false;
    // }

    auto url(get_url(URLQueryProcs));
    const auto& url = get_url(URLQueryProcs);
    if (url.empty()) {
        set_last_error("bus_query_procs url empty");
        return false;
@@ -221,6 +225,7 @@
    for(int i = 0; i < mtl.topic_list_size(); i ++){
        subscribe_topic(mtl.topic_list(i), &get<2>(*b));
    }
    std::move(mtl);

///////////////////////////////////////////////////
    auto ret = simple_request(url, topics, topics_len, reply, reply_len, timeout_ms);
@@ -283,6 +288,8 @@
    newPub.set_data(string{(const char*)msgpub, (const size_t)msgpub_len});

    string msg(newPub.SerializeAsString());
    std::move(newPub);

    auto ret = publish(pub.topic(), msg.data(), msg.size(), &get<1>(*b));
    if (ret > 0) return true;
    return false;
@@ -341,11 +348,7 @@
    string url{};
    // BHQueryTopicAddress获取proc_id
    string procid{};
    if (get_proc_id_from_MsgRequestTopic(request, request_len, timeout_ms, &procid)){
        // PRNTVITAG("get_proc_id_from_MsgRequestTopic failed");
        // return false;
        url = "ipc:///tmp/" + procid;
    }

    if (remote && remote_len > 0){
        BHAddress addr;
        if (addr.ParseFromArray(remote, remote_len)){
@@ -354,7 +357,12 @@
                // printf("======>> BHRequest use remote address %s\n", url.c_str());
            }
        }
    } else if (get_proc_id_from_MsgRequestTopic(request, request_len, timeout_ms, &procid)){
        // PRNTVITAG("get_proc_id_from_MsgRequestTopic failed");
        // return false;
        url = "ipc:///tmp/" + procid;
    }

    if (url.empty()) {
        set_last_error("bus_request url empty");
        return false;
@@ -383,13 +391,15 @@
    string msg;
    auto ret = read_request(src, &msg, timeout_ms, &get<6>(*b));
    // printf("bus_recv_request read_request ret %d msg %s\n", ret, msg.c_str());
    if (ret != 0) return false;
    if (ret < 0) return false;

    string procid{};
    if (get_proc_id_from_MsgRequestTopic(msg.data(), msg.size(), timeout_ms, &procid)){
        // PRNTVITAG("get_proc_id_from_MsgRequestTopic failed");
        // return false;
        copy_memory(proc_id, proc_id_len, procid.data(), procid.size());
    if (ret == REPLY_IPC){
        string procid{};
        if (get_proc_id_from_MsgRequestTopic(msg.data(), msg.size(), timeout_ms, &procid)){
            // PRNTVITAG("get_proc_id_from_MsgRequestTopic failed");
            // return false;
            copy_memory(proc_id, proc_id_len, procid.data(), procid.size());
        }
    }

    copy_memory(request, request_len, msg.data(), msg.size());