From: Adam Dickmeiss Date: Fri, 27 May 2005 18:00:28 +0000 (+0000) Subject: Fixed bug #355: YAZ proxy does not seem to load balance. Error X-Git-Tag: YAZPROXY.1.0.1~1 X-Git-Url: http://lists.indexdata.dk/?a=commitdiff_plain;h=6b6d0482e926e0e6de58ab13a2998cd93a4d6a58;p=yazproxy-moved-to-github.git Fixed bug #355: YAZ proxy does not seem to load balance. Error was caused by seemingly clever logic that choosed backend with most spares.. But sometimes the one with most spares is much more used. Not the backend with minimum use is always used. --- diff --git a/src/yaz-proxy.cpp b/src/yaz-proxy.cpp index 594435e..17ed69e 100644 --- a/src/yaz-proxy.cpp +++ b/src/yaz-proxy.cpp @@ -1,4 +1,4 @@ -/* $Id: yaz-proxy.cpp,v 1.26 2005-05-18 20:15:23 adam Exp $ +/* $Id: yaz-proxy.cpp,v 1.24.2.1 2005-05-27 18:00:28 adam Exp $ Copyright (c) 1998-2005, Index Data. This file is part of the yaz-proxy. @@ -119,7 +119,6 @@ Yaz_Proxy::Yaz_Proxy(IYaz_PDU_Observable *the_PDU_Observable, m_default_target = 0; m_proxy_negotiation_charset = 0; m_proxy_negotiation_lang = 0; - m_charset_converter = new Yaz_CharsetConverter; m_max_clients = 150; m_log_mask = 0; m_seed = time(0); @@ -199,7 +198,6 @@ Yaz_Proxy::~Yaz_Proxy() xfree(m_default_target); xfree(m_proxy_negotiation_charset); xfree(m_proxy_negotiation_lang); - delete m_charset_converter; xfree(m_optimize); #if HAVE_XSLT @@ -308,12 +306,12 @@ IYaz_PDU_Observer *Yaz_Proxy::sessionNotify(IYaz_PDU_Observable new_proxy->set_APDU_yazlog(1); else new_proxy->set_APDU_yazlog(0); + new_proxy->set_proxy_negotiation(m_proxy_negotiation_charset, + m_proxy_negotiation_lang); sprintf(new_proxy->m_session_str, "%ld:%d ", (long) time(0), m_session_no); m_session_no++; yaz_log (YLOG_LOG, "%sNew session %s", new_proxy->m_session_str, the_PDU_Observable->getpeername()); - new_proxy->set_proxy_negotiation(m_proxy_negotiation_charset, - m_proxy_negotiation_lang); return new_proxy; } @@ -390,12 +388,6 @@ const char *Yaz_Proxy::load_balance(const char **url) max_spare = zurl_in_spare[i]; } } - // use the one with minimum connections if spare is > 3 - if (spare_for_min > 3) - return ret_min; - // use one with most spares (if any) - if (max_spare > 0) - return ret_spare; return ret_min; } @@ -426,7 +418,6 @@ Yaz_ProxyClient *Yaz_Proxy::get_client(Z_APDU *apdu, const char *cookie, const char *cql2rpn_fname = 0; const char *negotiation_charset = 0; const char *negotiation_lang = 0; - const char *query_charset = 0; url[0] = m_default_target; url[1] = 0; if (cfg) @@ -441,8 +432,7 @@ Yaz_ProxyClient *Yaz_Proxy::get_client(Z_APDU *apdu, const char *cookie, &pre_init, &cql2rpn_fname, &negotiation_charset, - &negotiation_lang, - &query_charset); + &negotiation_lang); } if (client_idletime != -1) { @@ -453,10 +443,10 @@ Yaz_ProxyClient *Yaz_Proxy::get_client(Z_APDU *apdu, const char *cookie, m_cql2rpn.set_pqf_file(cql2rpn_fname); if (negotiation_charset || negotiation_lang) { + yaz_log(YLOG_LOG, "set_proxy_negotiation..."); set_proxy_negotiation(negotiation_charset, negotiation_lang); } - m_charset_converter->set_target_query_charset(query_charset); if (!url[0]) { yaz_log(YLOG_LOG, "%sNo default target", m_session_str); @@ -841,74 +831,6 @@ void Yaz_Proxy::convert_to_frontend_type(Z_NamePlusRecordList *p) } } -void Yaz_Proxy::convert_records_charset(Z_NamePlusRecordList *p, - const char *backend_charset) -{ - yaz_log(YLOG_LOG, "%sconvert_to_marc", m_session_str); - int sel = m_charset_converter->get_client_charset_selected(); - const char *client_record_charset = - m_charset_converter->get_client_query_charset(); - if (sel && backend_charset && client_record_charset && - strcmp(backend_charset, client_record_charset)) - { - int i; - yaz_iconv_t cd = yaz_iconv_open(client_record_charset, - backend_charset); - yaz_marc_t mt = yaz_marc_create(); - yaz_marc_xml(mt, YAZ_MARC_ISO2709); - yaz_marc_iconv(mt, cd); - for (i = 0; i < p->num_records; i++) - { - Z_NamePlusRecord *npr = p->records[i]; - if (npr->which == Z_NamePlusRecord_databaseRecord) - { - Z_External *r = npr->u.databaseRecord; - oident *ent = oid_getentbyoid(r->direct_reference); - if (!ent || ent->value == VAL_NONE) - continue; - - if (ent->value == VAL_SUTRS) - { - WRBUF w = wrbuf_alloc(); - - wrbuf_iconv_write(w, cd, (char*) r->u.octet_aligned->buf, - r->u.octet_aligned->len); - npr->u.databaseRecord = - z_ext_record(odr_encode(), ent->value, wrbuf_buf(w), - wrbuf_len(w)); - wrbuf_free(w, 1); - } - else if (ent->value == VAL_TEXT_XML) - { - ; - } - else if (r->which == Z_External_octet) - { - int rlen; - char *result; - if (yaz_marc_decode_buf(mt, - (char*) r->u.octet_aligned->buf, - r->u.octet_aligned->len, - &result, &rlen)) - { - npr->u.databaseRecord = - z_ext_record(odr_encode(), ent->value, result, rlen); - yaz_log(YLOG_LOG, "%sRecoding MARC record", - m_session_str); - } - } - } - } - if (cd) - yaz_iconv_close(cd); - yaz_marc_destroy(mt); - } - else - { - yaz_log(YLOG_LOG, "%sSkipping marc convert", m_session_str); - } -} - void Yaz_Proxy::convert_to_marcxml(Z_NamePlusRecordList *p, const char *backend_charset) { @@ -1267,9 +1189,6 @@ int Yaz_Proxy::send_to_client(Z_APDU *apdu) if (m_marcxml_mode == marcxml) convert_to_marcxml(p->u.databaseOrSurDiagnostics, m_backend_charset); - else - convert_records_charset(p->u.databaseOrSurDiagnostics, - m_backend_charset); if (convert_xsl(p->u.databaseOrSurDiagnostics, apdu)) return 0; @@ -1315,9 +1234,6 @@ int Yaz_Proxy::send_to_client(Z_APDU *apdu) if (m_marcxml_mode == marcxml) convert_to_marcxml(p->u.databaseOrSurDiagnostics, m_backend_charset); - else - convert_records_charset(p->u.databaseOrSurDiagnostics, - m_backend_charset); if (convert_xsl(p->u.databaseOrSurDiagnostics, apdu)) return 0; } @@ -1708,13 +1624,11 @@ void Yaz_Proxy::handle_charset_lang_negotiation(Z_APDU *apdu) { if (apdu->which == Z_APDU_initRequest) { - yaz_log(YLOG_LOG, "%shandle_charset_lang_negotiation", - m_session_str); if (m_initRequest_options && !ODR_MASK_GET(m_initRequest_options, Z_Options_negotiationModel) && (m_proxy_negotiation_charset || m_proxy_negotiation_lang)) { - // There is no negotiation proposal from + // There is not negotiation proposal from // client's side. OK. The proxy negotiation // in use, only. Z_InitRequest *initRequest = apdu->u.initRequest; @@ -1736,66 +1650,6 @@ void Yaz_Proxy::handle_charset_lang_negotiation(Z_APDU *apdu) 1); } } - else if (m_initRequest_options && - ODR_MASK_GET(m_initRequest_options, - Z_Options_negotiationModel) && - m_charset_converter->get_target_query_charset()) - { - yaz_log(YLOG_LOG, "%sManaged charset negotiation: charset=%s", - m_session_str, - m_charset_converter->get_target_query_charset()); - Z_InitRequest *initRequest = apdu->u.initRequest; - Z_CharSetandLanguageNegotiation *negotiation = - yaz_get_charneg_record (initRequest->otherInfo); - if (negotiation && - negotiation->which == Z_CharSetandLanguageNegotiation_proposal) - { - NMEM nmem = nmem_create(); - char **charsets = 0; - int num_charsets = 0; - char **langs = 0; - int num_langs = 0; - int selected = 0; - yaz_get_proposal_charneg (nmem, negotiation, - &charsets, &num_charsets, - &langs, &num_langs, &selected); - int i; - for (i = 0; iget_target_query_charset(); - // sweep through charsets and pick the first supported - // conversion - for (i = 0; iset_client_query_charset(c_charset); - m_charset_converter->set_client_charset_selected(selected); - } - nmem_destroy(nmem); - ODR_MASK_CLEAR(m_initRequest_options, - Z_Options_negotiationModel); - yaz_del_charneg_record(&initRequest->otherInfo); - } - else - { - yaz_log(YLOG_WARN, "%sUnable to decode charset package", - m_session_str); - } - } } else if (apdu->which == Z_APDU_initResponse) { @@ -1843,42 +1697,35 @@ void Yaz_Proxy::handle_charset_lang_negotiation(Z_APDU *apdu) ODR_MASK_CLEAR(initResponse->options, Z_Options_negotiationModel); // Delete negotiation (charneg-3) entry. - yaz_del_charneg_record(otherInfo); - } - } - else - { - if (m_proxy_negotiation_charset || m_proxy_negotiation_lang) - { - yaz_log(YLOG_LOG, "%sTarget did not honor negotiation", - m_session_str); - } - else if (m_charset_converter->get_client_query_charset()) - { - Z_OtherInformation **otherInfo; - Z_OtherInformationUnit *oi; - get_otherInfoAPDU(apdu, &otherInfo); - oi = update_otherInformation(otherInfo, 1, NULL, 0, 0); - if (oi) + Z_OtherInformation *p = *otherInfo; + for (int i=0; inum_elements; i++) { - ODR_MASK_SET(initResponse->options, - Z_Options_negotiationModel); - ODR_MASK_SET(m_initRequest_options, - Z_Options_negotiationModel); - - oi->which = Z_OtherInfo_externallyDefinedInfo; - oi->information.externallyDefinedInfo = - yaz_set_response_charneg( - odr_encode(), - m_charset_converter->get_client_query_charset(), - 0 /* no lang */, - m_charset_converter->get_client_charset_selected()); - } + if (p->list[i]->which == Z_OtherInfo_externallyDefinedInfo) + { + Z_External *pext = + p->list[i]->information.externallyDefinedInfo; + struct oident *e = oid_getentbyoid(pext->direct_reference); + + if (e && e->value == VAL_CHARNEG3 && e->oclass == CLASS_NEGOT && + pext->which == Z_External_charSetandLanguageNegotiation) + { + (p->num_elements)--; + if(p->num_elements == 0) + { + *otherInfo = 0; + } + else + { + for (int j=i; jnum_elements;j++) + p->list[j] = p->list[j+1]; + } + } + } + } } } } } - Z_Records *Yaz_Proxy::create_nonSurrogateDiagnostics(ODR odr, int error, const char *addinfo) @@ -1947,24 +1794,6 @@ Z_APDU *Yaz_Proxy::handle_query_transformation(Z_APDU *apdu) return apdu; } -Z_APDU *Yaz_Proxy::handle_target_charset_conversion(Z_APDU *apdu) -{ - if (apdu->which == Z_APDU_searchRequest && - apdu->u.searchRequest->query) - { - if (apdu->u.searchRequest->query->which == Z_Query_type_1 - || apdu->u.searchRequest->query->which == Z_Query_type_101) - { - if (m_http_version) - m_charset_converter->set_client_query_charset("UTF-8"); - Z_RPNQuery *rpnquery = apdu->u.searchRequest->query->u.type_1; - m_charset_converter->convert_type_1(rpnquery, odr_encode()); - } - } - return apdu; -} - - Z_APDU *Yaz_Proxy::handle_query_validation(Z_APDU *apdu) { if (apdu->which == Z_APDU_searchRequest) @@ -2232,6 +2061,7 @@ void Yaz_Proxy::srw_get_client(const char *db, const char **backend_db) int Yaz_Proxy::file_access(Z_HTTP_Request *hreq) { struct stat sbuf; + yaz_log(YLOG_LOG, "file_access"); if (strcmp(hreq->method, "GET")) return 0; if (hreq->path[0] != '/') @@ -2778,14 +2608,10 @@ void Yaz_Proxy::handle_incoming_Z_PDU(Z_APDU *apdu) apdu = handle_query_transformation(apdu); if (apdu) - apdu = handle_target_charset_conversion(apdu); - - if (apdu) apdu = handle_query_validation(apdu); if (apdu) apdu = result_set_optimize(apdu); - if (!apdu) { m_client->timeout(m_target_idletime); // mark it active even @@ -3000,8 +2826,7 @@ void Yaz_Proxy::pre_init() &cql2rpn, &authentication, &negotiation_charset, - &negotiation_lang, - 0) ; i++) + &negotiation_lang) ; i++) { if (pre_init) {