-/* $Id: filter_z3950_client.cpp,v 1.5 2005-10-25 21:32:01 adam Exp $
+/* $Id: filter_z3950_client.cpp,v 1.9 2005-10-30 18:51:21 adam Exp $
Copyright (c) 2005, Index Data.
%LICENSE%
#include "package.hpp"
#include <boost/thread/mutex.hpp>
+#include <boost/thread/condition.hpp>
+#include "util.hpp"
#include "filter_z3950_client.hpp"
#include <yaz/zgdu.h>
namespace yp2 {
namespace filter {
class Z3950Client::Assoc : public yazpp_1::Z_Assoc{
- friend class Pimpl;
+ friend class Rep;
public:
Assoc(yazpp_1::SocketManager *socket_manager,
yazpp_1::IPDU_Observable *PDU_Observable,
yazpp_1::SocketManager *m_socket_manager;
yazpp_1::IPDU_Observable *m_PDU_Observable;
Package *m_package;
+ bool m_in_use;
bool m_waiting;
bool m_connected;
std::string m_host;
};
- class Z3950Client::Pimpl {
+ class Z3950Client::Rep {
public:
boost::mutex m_mutex;
+ boost::condition m_cond_session_ready;
std::map<yp2::Session,Z3950Client::Assoc *> m_clients;
Z3950Client::Assoc *get_assoc(Package &package);
void send_and_receive(Package &package,
yf::Z3950Client::Assoc *c);
- void release_assoc(Package &package,
- yf::Z3950Client::Assoc *c);
+ void release_assoc(Package &package);
};
}
}
std::string host)
: Z_Assoc(PDU_Observable),
m_socket_manager(socket_manager), m_PDU_Observable(PDU_Observable),
- m_package(0), m_waiting(false), m_connected(false),
+ m_package(0), m_in_use(true), m_waiting(false), m_connected(false),
m_host(host)
{
// std::cout << "create assoc " << this << "\n";
{
m_waiting = false;
- ODR odr = odr_createmem(ODR_ENCODE);
-
- Z_APDU *apdu = zget_APDU(odr, Z_APDU_close);
-
- *apdu->u.close->closeReason = Z_Close_peerAbort;
+ yp2::odr odr;
if (m_package)
{
- m_package->response() = apdu;
+ m_package->response() = odr.create_close(Z_Close_peerAbort, 0);
m_package->session().close();
}
-
- odr_destroy(odr);
}
void yf::Z3950Client::Assoc::timeoutNotify()
{
m_waiting = false;
- ODR odr = odr_createmem(ODR_ENCODE);
-
- Z_APDU *apdu = zget_APDU(odr, Z_APDU_close);
-
- *apdu->u.close->closeReason = Z_Close_lackOfActivity;
+ yp2::odr odr;
if (m_package)
{
- m_package->response() = apdu;
+ m_package->response() = odr.create_close(Z_Close_lackOfActivity, 0);
m_package->session().close();
}
- odr_destroy(odr);
}
void yf::Z3950Client::Assoc::recv_GDU(Z_GDU *gdu, int len)
}
-yf::Z3950Client::Z3950Client() {
- m_p = new yf::Z3950Client::Pimpl;
+yf::Z3950Client::Z3950Client() : m_p(new yf::Z3950Client::Rep)
+{
}
yf::Z3950Client::~Z3950Client() {
- delete m_p;
}
-yf::Z3950Client::Assoc *yf::Z3950Client::Pimpl::get_assoc(Package &package)
+yf::Z3950Client::Assoc *yf::Z3950Client::Rep::get_assoc(Package &package)
{
// only one thread messes with the clients list at a time
boost::mutex::scoped_lock lock(m_mutex);
std::map<yp2::Session,yf::Z3950Client::Assoc *>::iterator it;
- it = m_clients.find(package.session());
- if (it != m_clients.end())
- return it->second;
+ while(true)
+ {
+ it = m_clients.find(package.session());
+ if (it == m_clients.end())
+ break;
+
+ if (!it->second->m_in_use)
+ {
+ it->second->m_in_use = true;
+ return it->second;
+ }
+ m_cond_session_ready.wait(lock);
+ }
// only deal with Z39.50
Z_GDU *gdu = package.request().get();
// check that it is init. If not, close
if (apdu->which != Z_APDU_initRequest)
{
- ODR odr = odr_createmem(ODR_ENCODE);
- Z_APDU *apdu = zget_APDU(odr, Z_APDU_close);
-
- *apdu->u.close->closeReason = Z_Close_protocolError;
- apdu->u.close->diagnosticInformation =
- odr_strdup(odr, "no init request for session");
-
- package.response() = apdu;
+ yp2::odr odr;
+ package.response() = odr.create_close(Z_Close_protocolError,
+ "no init request for session");
package.session().close();
- odr_destroy(odr);
return 0;
}
// check virtual host
VAL_PROXY, 1, 0);
if (!vhost)
{
- ODR odr = odr_createmem(ODR_ENCODE);
- Z_APDU *apdu = zget_APDU(odr, Z_APDU_initResponse);
+ yp2::odr odr;
+ package.response() = odr.create_initResponse(
+ YAZ_BIB1_INIT_NEGOTIATION_OPTION_REQUIRED,
+ "Virtual host not given");
- apdu->u.initResponse->userInformationField =
- zget_init_diagnostics(odr,
- YAZ_BIB1_INIT_NEGOTIATION_OPTION_REQUIRED,
- "Virtual host not given");
- package.response() = apdu;
-
package.session().close();
- odr_destroy(odr);
return 0;
}
return as;
}
-void yf::Z3950Client::Pimpl::send_and_receive(Package &package,
- yf::Z3950Client::Assoc *c)
+void yf::Z3950Client::Rep::send_and_receive(Package &package,
+ yf::Z3950Client::Assoc *c)
{
Z_GDU *gdu = package.request().get();
if (!gdu || gdu->which != Z_GDU_Z3950)
return;
- // we should lock c!
-
c->m_package = &package;
c->m_waiting = true;
if (!c->m_connected)
;
}
-void yf::Z3950Client::Pimpl::release_assoc(Package &package,
- yf::Z3950Client::Assoc *c)
+void yf::Z3950Client::Rep::release_assoc(Package &package)
{
- if (package.session().is_closed())
+ boost::mutex::scoped_lock lock(m_mutex);
+ std::map<yp2::Session,yf::Z3950Client::Assoc *>::iterator it;
+
+ it = m_clients.find(package.session());
+ if (it != m_clients.end())
{
- boost::mutex::scoped_lock lock(m_mutex);
- std::map<yp2::Session,yf::Z3950Client::Assoc *>::iterator it;
-
- it = m_clients.find(package.session());
- if (it != m_clients.end())
+ if (package.session().is_closed())
{
// the Z_Assoc and PDU_Assoc must be destroyed before
// the socket manager.. so pull that out.. first..
delete s; // then manager
m_clients.erase(it);
}
+ else
+ {
+ it->second->m_in_use = false;
+ }
+ m_cond_session_ready.notify_all();
}
}
if (c)
{
m_p->send_and_receive(package, c);
- m_p->release_assoc(package, c);
}
+ m_p->release_assoc(package);
}