Tcp client ne renvoie pas d'erreur lors de la connexion à localhost si aucun serveur n'est prêt

Question 1

J'ai essayé d'englober toutes les fonctionnalités de réseautage pour mon programme à l'intérieur d'un "SimpleClient" de la classe, qui expose cette interface à l'intérieur de l' .php fichier:

class SimpleClient{
    //private
        boost::shared_ptr<boost::asio::ip::tcp::socket> signal_socket; 
    protected: 
        boost::asio::io_context my_context;
        ClientState state; //0: Ready to be used, not connected,  -1: error, 1: Connected/active, 
    public: 
       SimpleClient();
       virtual bool connect(const char* ip_address);
       virtual void disconnect();
       virtual bool sendMessage(const char* msg, int length);
       virtual int getResponse( char* msg, int length, int timeout);
       virtual int getSignalData( char* msg, int length);
       virtual ClientState getState();
};

Lors du test de la méthode de connexion, j'ai remarqué que, même si aucun serveur n'a été en attente sur localhost sur le port 8887, la connexion ne jetez pas une erreur. Comment puis-je vérifier que la prise est connectée pour de vrai?

Voici les méthodes que j'ai mis en place et un petit exemple qui reproduit mon comportement, à l'aide de coup de pouce.test:

SimpleClient::SimpleClient() : my_context() {
    signal_socket.reset();
    state = ClientState::CL_UNCON;
}

bool SimpleClient::connect(const char* ip_address) {
    boost::system::error_code ec;
    if (signal_socket != NULL && signal_socket->is_open() )
        return true;
    try {
        boost::asio::ip::tcp::endpoint signal_endpoint(boost::asio::ip::make_address(ip_address), 8887);
        signal_socket.reset(new boost::asio::ip::tcp::socket(my_context));
        signal_socket->connect(signal_endpoint, ec);
        if(ec) {
            std::cout<<ec.message()<<" "<<ec.value()<<std::endl;
            return false;
        }
    } catch(const boost::system::system_error& ex) {
        std::cout<<ex.code()<<std::endl;
        return false;
    }
    std::cout<<signal_socket<<std::endl;
    state = ClientState::CL_READY;
    return true;
}

Voici le programme de test:

#define BOOST_AUTO_TEST_MAIN 
#define BOOST_TEST_DYN_LINK

#include "SimpleClient.hpp"

#include <boost/test/unit_test.hpp>

#include <string.h>

#define TARGET_IP "127.0.0.1"
#define BAD_IP "128.0.0.1"


BOOST_AUTO_TEST_CASE(connection_test)  {
    bool ret;
    std::cout<<"Connect to WRONG address"<<std::endl;
    SimpleClient* ut = new SimpleClient();
    BOOST_CHECK_EQUAL(ut->getState(), ClientState::CL_UNCON);
    ret = ut->connect(BAD_IP);
    BOOST_CHECK(!ret);
    std::cout<<"Connect to RIGHT address"<<std::endl;
    ret = ut->connect(TARGET_IP);
    BOOST_CHECK(ret);
    ut->disconnect();
    try{
        delete ut;
    }
    catch(const std::exception& ex) {
        std::cout<<ex.what()<<std::endl;
    }
    catch(...) {
    }
}

Question 2

Le problème était à l'intérieur de ma méthode de connexion, évidemment. Le premier appel à la connecter à l'intérieur de mon test a échoué, mais il n'a pas fermer le socket de sorte que le deuxième appel automatiquement renvoyé true sans effectuer aucune opération.

TLDR: en cas de panne, vous connecter ne pas fermer le socket, et tout appel à is_open() retourne 'true', appeler manuellement la méthode close() sur le socket, en cas d'erreur.

bool SimpleClient::connect(const char* ip_address) {
    boost::system::error_code ec;
    if (signal_socket != NULL && signal_socket->is_open() )
        return true;
    try {
        boost::asio::ip::tcp::endpoint signal_endpoint(boost::asio::ip::make_address(ip_address), 8887);
        signal_socket.reset(new boost::asio::ip::tcp::socket(my_context));
        signal_socket->connect(signal_endpoint, ec);
        if(ec) {
            std::cout<<ec.message()<<" "<<ec.value()<<std::endl;
            signal_socket->close();
            signal_socket.reset();
            return false;
        }
    } catch(const boost::system::system_error& ex) {
        std::cout<<ex.code()<<std::endl;
        signal_socket->close();
        signal_socket.reset();
        return false;
    }
    std::cout<<signal_socket<<std::endl;
    state = ClientState::CL_READY;
    return true;
}

Gionata Benelli · Answer 1 · 2021-11-22T14:52:56

Le problème était à l'intérieur de ma méthode de connexion, évidemment. Le premier appel à la connecter à l'intérieur de mon test a échoué, mais il n'a pas fermer le socket de sorte que le deuxième appel automatiquement renvoyé true sans effectuer aucune opération.

TLDR: en cas de panne, vous connecter ne pas fermer le socket, et tout appel à is_open() retourne 'true', appeler manuellement la méthode close() sur le socket, en cas d'erreur.

bool SimpleClient::connect(const char* ip_address) {
    boost::system::error_code ec;
    if (signal_socket != NULL && signal_socket->is_open() )
        return true;
    try {
        boost::asio::ip::tcp::endpoint signal_endpoint(boost::asio::ip::make_address(ip_address), 8887);
        signal_socket.reset(new boost::asio::ip::tcp::socket(my_context));
        signal_socket->connect(signal_endpoint, ec);
        if(ec) {
            std::cout<<ec.message()<<" "<<ec.value()<<std::endl;
            signal_socket->close();
            signal_socket.reset();
            return false;
        }
    } catch(const boost::system::system_error& ex) {
        std::cout<<ex.code()<<std::endl;
        signal_socket->close();
        signal_socket.reset();
        return false;
    }
    std::cout<<signal_socket<<std::endl;
    state = ClientState::CL_READY;
    return true;
}

Sinon ajuster le flux de contrôle, de sorte que la prise n'est pas utilisée après une erreur. C'est la plus naturelle. Souvent, la vérification de la "is_open" n'est pas ce que vous êtes après de toute façon (par exemple, il ne sera pas en mesure de détecter le moment où le distant a fermé la connexion). +1 pour répondre à votre propre question!

Tcp client ne renvoie pas d'erreur lors de la connexion à localhost si aucun serveur n'est prêt

La question

La meilleure réponse

Dans d'autres langues

Cette page est dans d'autres langues

Populaires dans cette catégorie

Questions populaires dans cette catégorie