Changed the SRU update structures and codecs to reflect the SRU pre 1.0
[yaz-moved-to-github.git] / src / seshigh.c
index 2a3fc2f..a91c70a 100644 (file)
@@ -1,8 +1,8 @@
 /*
- * Copyright (C) 1995-2005, Index Data ApS
+ * Copyright (C) 1995-2006, Index Data ApS
  * See the file LICENSE for details.
  *
- * $Id: seshigh.c,v 1.96 2006-08-24 13:25:45 adam Exp $
+ * $Id: seshigh.c,v 1.106 2006-12-06 21:35:58 adam Exp $
  */
 /**
  * \file seshigh.c
@@ -108,7 +108,7 @@ static int log_request = 0; /* one-line logs for requests */
 static int log_requestdetail = 0;  /* more detailed stuff */
 
 /** get_logbits sets global loglevel bits */
-static void get_logbits()
+static void get_logbits(void)
 { /* needs to be called after parsing cmd-line args that can set loglevels!*/
     if (!logbits_set)
     {
@@ -264,6 +264,98 @@ static void do_close(association *a, int reason, char *message)
     do_close_req (a, reason, message, req);
 }
 
+
+int ir_read(IOCHAN h, int event)
+{
+    association *assoc = (association *)iochan_getdata(h);
+    COMSTACK conn = assoc->client_link;
+    request *req;
+    
+    if ((assoc->cs_put_mask & EVENT_INPUT) == 0 && (event & assoc->cs_get_mask))
+    {
+        yaz_log(YLOG_DEBUG, "ir_session (input)");
+        /* We aren't speaking to this fellow */
+        if (assoc->state == ASSOC_DEAD)
+        {
+            yaz_log(log_sessiondetail, "Connection closed - end of session");
+            cs_close(conn);
+            destroy_association(assoc);
+            iochan_destroy(h);
+            return 0;
+        }
+        assoc->cs_get_mask = EVENT_INPUT;
+
+        do
+        {
+            int res = cs_get(conn, &assoc->input_buffer,
+                             &assoc->input_buffer_len);
+            if (res < 0 && cs_errno(conn) == CSBUFSIZE)
+            {
+                yaz_log(log_session, "Connection error: %s res=%d",
+                        cs_errmsg(cs_errno(conn)), res);
+                req = request_get(&assoc->incoming); /* get a new request */
+                do_close_req(assoc, Z_Close_protocolError, 
+                             "Incoming package too large", req);
+                return 0;
+            }
+            else if (res <= 0)
+            {
+                yaz_log(log_session, "Connection closed by client");
+                assoc->state = ASSOC_DEAD;
+                return 0;
+            }
+            else if (res == 1) /* incomplete read - wait for more  */
+            {
+                if (conn->io_pending & CS_WANT_WRITE)
+                    assoc->cs_get_mask |= EVENT_OUTPUT;
+                iochan_setflag(h, assoc->cs_get_mask);
+                return 1;
+            }
+            /* we got a complete PDU. Let's decode it */
+            yaz_log(YLOG_DEBUG, "Got PDU, %d bytes: lead=%02X %02X %02X", res,
+                    assoc->input_buffer[0] & 0xff,
+                    assoc->input_buffer[1] & 0xff,
+                    assoc->input_buffer[2] & 0xff);
+            req = request_get(&assoc->incoming); /* get a new request */
+            odr_reset(assoc->decode);
+            odr_setbuf(assoc->decode, assoc->input_buffer, res, 0);
+            if (!z_GDU(assoc->decode, &req->gdu_request, 0, 0))
+            {
+                yaz_log(YLOG_WARN, "ODR error on incoming PDU: %s [element %s] "
+                        "[near byte %ld] ",
+                        odr_errmsg(odr_geterror(assoc->decode)),
+                        odr_getelement(assoc->decode),
+                        (long) odr_offset(assoc->decode));
+                if (assoc->decode->error != OHTTP)
+                {
+                    yaz_log(YLOG_WARN, "PDU dump:");
+                    odr_dumpBER(yaz_log_file(), assoc->input_buffer, res);
+                    request_release(req);
+                    do_close(assoc, Z_Close_protocolError, "Malformed package");
+                }
+                else
+                {
+                    Z_GDU *p = z_get_HTTP_Response(assoc->encode, 400);
+                    assoc->state = ASSOC_DEAD;
+                    process_gdu_response(assoc, req, p);
+                }
+                return 0;
+            }
+            req->request_mem = odr_extract_mem(assoc->decode);
+            if (assoc->print) 
+            {
+                if (!z_GDU(assoc->print, &req->gdu_request, 0, 0))
+                    yaz_log(YLOG_WARN, "ODR print error: %s", 
+                            odr_errmsg(odr_geterror(assoc->print)));
+                odr_reset(assoc->print);
+            }
+            request_enq(&assoc->incoming, req);
+        }
+        while (cs_more(conn));
+    }
+    return 1;
+}
+
 /*
  * This is where PDUs from the client are read and the further
  * processing is initiated. Flow of control moves down through the
@@ -324,91 +416,10 @@ void ir_session(IOCHAN h, int event)
         }
         return;
     }
-    if ((event & assoc->cs_get_mask) || (event & EVENT_WORK)) /* input */
+    if (event & assoc->cs_get_mask) /* input */
     {
-        if ((assoc->cs_put_mask & EVENT_INPUT) == 0 && (event & assoc->cs_get_mask))
-        {
-            yaz_log(YLOG_DEBUG, "ir_session (input)");
-            /* We aren't speaking to this fellow */
-            if (assoc->state == ASSOC_DEAD)
-            {
-                yaz_log(log_sessiondetail, "Connection closed - end of session");
-                cs_close(conn);
-                destroy_association(assoc);
-                iochan_destroy(h);
-                return;
-            }
-            assoc->cs_get_mask = EVENT_INPUT;
-            if ((res = cs_get(conn, &assoc->input_buffer,
-                &assoc->input_buffer_len)) == 0)
-            {
-                yaz_log(log_sessiondetail, "Connection closed by client");
-                cs_close(conn);
-                destroy_association(assoc);
-                iochan_destroy(h);
-                return;
-            }
-            else if (res < 0)
-            {
-                yaz_log(log_session, "Connection error: %s",
-                        cs_errmsg(cs_errno(conn)));
-                req = request_get(&assoc->incoming); /* get a new request */
-                do_close_req(assoc, Z_Close_protocolError, 
-                             "Incoming package too large", req);
-                return;
-            }
-            else if (res == 1) /* incomplete read - wait for more  */
-            {
-                if (conn->io_pending & CS_WANT_WRITE)
-                    assoc->cs_get_mask |= EVENT_OUTPUT;
-                iochan_setflag(h, assoc->cs_get_mask);
-                return;
-            }
-            if (cs_more(conn)) /* more stuff - call us again later, please */
-                iochan_setevent(h, EVENT_INPUT);
-                
-            /* we got a complete PDU. Let's decode it */
-            yaz_log(YLOG_DEBUG, "Got PDU, %d bytes: lead=%02X %02X %02X", res,
-                            assoc->input_buffer[0] & 0xff,
-                            assoc->input_buffer[1] & 0xff,
-                            assoc->input_buffer[2] & 0xff);
-            req = request_get(&assoc->incoming); /* get a new request */
-            odr_reset(assoc->decode);
-            odr_setbuf(assoc->decode, assoc->input_buffer, res, 0);
-            if (!z_GDU(assoc->decode, &req->gdu_request, 0, 0))
-            {
-                yaz_log(YLOG_WARN, "ODR error on incoming PDU: %s [element %s] "
-                        "[near byte %ld] ",
-                        odr_errmsg(odr_geterror(assoc->decode)),
-                        odr_getelement(assoc->decode),
-                        (long) odr_offset(assoc->decode));
-                if (assoc->decode->error != OHTTP)
-                {
-                    yaz_log(YLOG_WARN, "PDU dump:");
-                    odr_dumpBER(yaz_log_file(), assoc->input_buffer, res);
-                    request_release(req);
-                    do_close(assoc, Z_Close_protocolError, "Malformed package");
-                }
-                else
-                {
-                    Z_GDU *p = z_get_HTTP_Response(assoc->encode, 400);
-                    assoc->state = ASSOC_DEAD;
-                    process_gdu_response(assoc, req, p);
-                }
-                return;
-            }
-            req->request_mem = odr_extract_mem(assoc->decode);
-            if (assoc->print) 
-            {
-                if (!z_GDU(assoc->print, &req->gdu_request, 0, 0))
-                    yaz_log(YLOG_WARN, "ODR print error: %s", 
-                       odr_errmsg(odr_geterror(assoc->print)));
-                odr_reset(assoc->print);
-            }
-            request_enq(&assoc->incoming, req);
-        }
-
-        /* can we do something yet? */
+        if (!ir_read(h, event))
+            return;
         req = request_head(&assoc->incoming);
         if (req->state == REQUEST_IDLE)
         {
@@ -518,9 +529,6 @@ static int srw_bend_init(association *assoc, Z_SRW_diagnostic **d, int *num, Z_S
         yaz_log(log_requestdetail, "srw_bend_init config=%s", cb->configname);
         assoc_init_reset(assoc);
         
-        assoc->maximumRecordSize = 3000000;
-        assoc->preferredMessageSize = 3000000;
-
         if (sr->username)
         {
             Z_IdAuthentication *auth = odr_malloc(assoc->decode, sizeof(*auth));
@@ -567,33 +575,6 @@ static int srw_bend_init(association *assoc, Z_SRW_diagnostic **d, int *num, Z_S
     return 1;
 }
 
-static const char *get_esn(Z_RecordComposition *comp)
-{
-    if (comp && comp->which == Z_RecordComp_complex)
-    {
-        if (comp->u.complex->generic 
-            && comp->u.complex->generic->elementSpec
-            && (comp->u.complex->generic->elementSpec->which == 
-                Z_ElementSpec_elementSetName))
-            return comp->u.complex->generic->elementSpec->u.elementSetName;
-    }
-    else if (comp && comp->which == Z_RecordComp_simple &&
-             comp->u.simple->which == Z_ElementSetNames_generic)
-        return comp->u.simple->u.generic;
-    return 0;
-}
-
-static void set_esn(Z_RecordComposition **comp_p, const char *esn, NMEM nmem)
-{
-    Z_RecordComposition *comp = nmem_malloc(nmem, sizeof(*comp));
-    
-    comp->which = Z_RecordComp_simple;
-    comp->u.simple = nmem_malloc(nmem, sizeof(*comp->u.simple));
-    comp->u.simple->which = Z_ElementSetNames_generic;
-    comp->u.simple->u.generic = nmem_strdup(nmem, esn);
-    *comp_p = comp;
-}
-
 static int retrieve_fetch(association *assoc, bend_fetch_rr *rr)
 {
 #if YAZ_HAVE_XML2
@@ -604,7 +585,7 @@ static int retrieve_fetch(association *assoc, bend_fetch_rr *rr)
     if (assoc->server)
     {
         int r;
-        const char *input_schema = get_esn(rr->comp);
+        const char *input_schema = yaz_get_esn(rr->comp);
         Odr_oid *input_syntax_raw = rr->request_format_raw;
         
         const char *backend_schema = 0;
@@ -649,7 +630,7 @@ static int retrieve_fetch(association *assoc, bend_fetch_rr *rr)
         }
         if (backend_schema)
         {
-            set_esn(&rr->comp, backend_schema, rr->stream->mem);
+            yaz_set_esn(&rr->comp, backend_schema, rr->stream->mem);
         }
         if (backend_syntax)
         {
@@ -1015,114 +996,113 @@ static void srw_bend_search(association *assoc, request *req,
                     srw_res->resultSetIdleTime =
                         odr_intdup(assoc->encode, *rr.srw_setnameIdleTime );
                }
-                if (number > 0)
+                
+                if ((rr.hits > 0 && start > rr.hits) || start < 1)
+                {
+                    yaz_add_srw_diagnostic(
+                        assoc->encode, 
+                        &srw_res->diagnostics, &srw_res->num_diagnostics,
+                        YAZ_SRW_FIRST_RECORD_POSITION_OUT_OF_RANGE, 0);
+                }
+                else if (number > 0)
                 {
                     int i;
+                    int ok = 1;
+                    if (start + number > rr.hits)
+                        number = rr.hits - start + 1;
                     
-                    if (start > rr.hits)
+                    /* Call bend_present if defined */
+                    if (assoc->init->bend_present)
                     {
-                        yaz_add_srw_diagnostic(assoc->encode, &srw_res->diagnostics,
-                                               &srw_res->num_diagnostics,
-                                               YAZ_SRW_FIRST_RECORD_POSITION_OUT_OF_RANGE, 0);
+                        bend_present_rr *bprr = (bend_present_rr*)
+                            odr_malloc (assoc->decode, sizeof(*bprr));
+                        bprr->setname = "default";
+                        bprr->start = start;
+                        bprr->number = number;
+                        bprr->format = VAL_TEXT_XML;
+                        if (srw_req->recordSchema)
+                        {
+                            bprr->comp = (Z_RecordComposition *) odr_malloc(assoc->decode,
+                                                                            sizeof(*bprr->comp));
+                            bprr->comp->which = Z_RecordComp_simple;
+                            bprr->comp->u.simple = (Z_ElementSetNames *)
+                                odr_malloc(assoc->decode, sizeof(Z_ElementSetNames));
+                            bprr->comp->u.simple->which = Z_ElementSetNames_generic;
+                            bprr->comp->u.simple->u.generic = srw_req->recordSchema;
+                        }
+                        else
+                        {
+                            bprr->comp = 0;
+                        }
+                        bprr->stream = assoc->encode;
+                        bprr->referenceId = 0;
+                        bprr->print = assoc->print;
+                        bprr->request = req;
+                        bprr->association = assoc;
+                        bprr->errcode = 0;
+                        bprr->errstring = NULL;
+                        (*assoc->init->bend_present)(assoc->backend, bprr);
+                        
+                        if (!bprr->request)
+                            return;
+                        if (bprr->errcode)
+                        {
+                            srw_error = yaz_diag_bib1_to_srw (bprr->errcode);
+                            yaz_add_srw_diagnostic(assoc->encode,
+                                                   &srw_res->diagnostics,
+                                                   &srw_res->num_diagnostics,
+                                                   srw_error, bprr->errstring);
+                            ok = 0;
+                        }
                     }
-                    else
+                    
+                    if (ok)
                     {
-                        int ok = 1;
-                        if (start + number > rr.hits)
-                            number = rr.hits - start + 1;
-
-                        /* Call bend_present if defined */
-                        if (assoc->init->bend_present)
+                        int j = 0;
+                        int packing = Z_SRW_recordPacking_string;
+                        if (srw_req->recordPacking)
                         {
-                            bend_present_rr *bprr = (bend_present_rr*)
-                                odr_malloc (assoc->decode, sizeof(*bprr));
-                            bprr->setname = "default";
-                            bprr->start = start;
-                            bprr->number = number;
-                            bprr->format = VAL_TEXT_XML;
-                            if (srw_req->recordSchema)
-                            {
-                                bprr->comp = (Z_RecordComposition *) odr_malloc(assoc->decode,
-                                        sizeof(*bprr->comp));
-                                bprr->comp->which = Z_RecordComp_simple;
-                                bprr->comp->u.simple = (Z_ElementSetNames *)
-                                    odr_malloc(assoc->decode, sizeof(Z_ElementSetNames));
-                                bprr->comp->u.simple->which = Z_ElementSetNames_generic;
-                                bprr->comp->u.simple->u.generic = srw_req->recordSchema;
-                            }
-                            else
-                            {
-                                bprr->comp = 0;
-                            }
-                            bprr->stream = assoc->encode;
-                            bprr->referenceId = 0;
-                            bprr->print = assoc->print;
-                            bprr->request = req;
-                            bprr->association = assoc;
-                            bprr->errcode = 0;
-                            bprr->errstring = NULL;
-                            (*assoc->init->bend_present)(assoc->backend, bprr);
+                            packing = 
+                                yaz_srw_str_to_pack(srw_req->recordPacking);
+                            if (packing == -1)
+                                packing = Z_SRW_recordPacking_string;
+                        }
+                        srw_res->records = (Z_SRW_record *)
+                            odr_malloc(assoc->encode,
+                                       number * sizeof(*srw_res->records));
+                        
+                        srw_res->extra_records = (Z_SRW_extra_record **)
+                            odr_malloc(assoc->encode,
+                                       number*sizeof(*srw_res->extra_records));
+
+                        for (i = 0; i<number; i++)
+                        {
+                            int errcode;
+                            const char *addinfo = 0;
                             
-                            if (!bprr->request)
-                                return;
-                            if (bprr->errcode)
+                            srw_res->records[j].recordPacking = packing;
+                            srw_res->records[j].recordData_buf = 0;
+                            srw_res->extra_records[j] = 0;
+                            yaz_log(YLOG_DEBUG, "srw_bend_fetch %d", i+start);
+                            errcode = srw_bend_fetch(assoc, i+start, srw_req,
+                                                     srw_res->records + j,
+                                                     &addinfo);
+                            if (errcode)
                             {
-                                srw_error = yaz_diag_bib1_to_srw (bprr->errcode);
                                 yaz_add_srw_diagnostic(assoc->encode,
                                                        &srw_res->diagnostics,
                                                        &srw_res->num_diagnostics,
-                                                       srw_error, bprr->errstring);
-                                ok = 0;
-                            }
-                        }
-
-                        if (ok)
-                        {
-                            int j = 0;
-                            int packing = Z_SRW_recordPacking_string;
-                            if (srw_req->recordPacking){
-                                if (!strcmp(srw_req->recordPacking, "xml"))
-                                    packing = Z_SRW_recordPacking_XML;
-                                if (!strcmp(srw_req->recordPacking, "url"))
-                                    packing = Z_SRW_recordPacking_URL;
-                            }
-                            srw_res->records = (Z_SRW_record *)
-                                odr_malloc(assoc->encode,
-                                           number * sizeof(*srw_res->records));
-                            
-                            srw_res->extra_records = (Z_SRW_extra_record **)
-                                odr_malloc(assoc->encode,
-                                           number*sizeof(*srw_res->extra_records));
-
-                            for (i = 0; i<number; i++)
-                            {
-                                int errcode;
-                                const char *addinfo = 0;
+                                                       yaz_diag_bib1_to_srw (errcode),
+                                                       addinfo);
                                 
-                                srw_res->records[j].recordPacking = packing;
-                                srw_res->records[j].recordData_buf = 0;
-                                srw_res->extra_records[j] = 0;
-                                yaz_log(YLOG_DEBUG, "srw_bend_fetch %d", i+start);
-                                errcode = srw_bend_fetch(assoc, i+start, srw_req,
-                                                         srw_res->records + j,
-                                                         &addinfo);
-                                if (errcode)
-                                {
-                                    yaz_add_srw_diagnostic(assoc->encode,
-                                                           &srw_res->diagnostics,
-                                                           &srw_res->num_diagnostics,
-                                                           yaz_diag_bib1_to_srw (errcode),
-                                                           addinfo);
-                                    
-                                    break;
-                                }
-                                if (srw_res->records[j].recordData_buf)
-                                    j++;
+                                break;
                             }
-                            srw_res->num_records = j;
-                            if (!j)
-                                srw_res->records = 0;
+                            if (srw_res->records[j].recordData_buf)
+                                j++;
                         }
+                        srw_res->num_records = j;
+                        if (!j)
+                            srw_res->records = 0;
                     }
                 }
             }
@@ -1237,10 +1217,10 @@ static void srw_bend_explain(association *assoc, request *req,
             int packing = Z_SRW_recordPacking_string;
             if (srw_req->recordPacking)
             {
-                if (!strcmp(srw_req->recordPacking, "xml"))
-                    packing = Z_SRW_recordPacking_XML;
-                else if (!strcmp(srw_req->recordPacking, "url"))
-                    packing = Z_SRW_recordPacking_URL;
+                packing = 
+                    yaz_srw_str_to_pack(srw_req->recordPacking);
+                if (packing == -1)
+                    packing = Z_SRW_recordPacking_string;
             }
             srw_res->record.recordSchema = rr.schema;
             srw_res->record.recordPacking = packing;
@@ -1453,13 +1433,14 @@ static void srw_bend_update(association *assoc, request *req,
                            int *http_code)
 {
     Z_SRW_updateRequest *srw_req = sr->u.update_request;
-    yaz_log(YLOG_DEBUG, "Got SRW UpdateRequest");
+    yaz_log(log_session, "SRWUpdate action=%s", srw_req->operation);
     yaz_log(YLOG_DEBUG, "num_diag = %d", srw_res->num_diagnostics );
     *http_code = 404;
     srw_bend_init(assoc, &srw_res->diagnostics, &srw_res->num_diagnostics, sr);
     if (assoc->init)
     {
        bend_update_rr rr;
+        Z_SRW_extra_record *extra = srw_req->extra_record;
        
        rr.stream = assoc->encode;
        rr.print = assoc->print;
@@ -1468,143 +1449,169 @@ static void srw_bend_update(association *assoc, request *req,
        rr.operation = srw_req->operation;
        rr.operation_status = "failed";
        rr.record_id = 0;
-        rr.record_version = 0;
-        rr.record_checksum = 0;
-        rr.record_old_version = 0;
-        rr.record_packing = "xml";
+        rr.record_versions = 0;
+        rr.num_versions = 0;
+        rr.record_packing = "string";
        rr.record_schema = 0;
         rr.record_data = 0;
-        rr.request_extra_record = 0;
-        rr.response_extra_record = 0;
+        rr.extra_record_data = 0;
         rr.extra_request_data = 0;
         rr.extra_response_data = 0;
         rr.uri = 0;
         rr.message = 0;
         rr.details = 0;
-
+        
+       *http_code = 200;
+        if (rr.operation == 0)
+        {
+            yaz_add_sru_update_diagnostic(
+                assoc->encode, &srw_res->diagnostics,
+                &srw_res->num_diagnostics,
+                YAZ_SRU_UPDATE_MISSING_MANDATORY_ELEMENT_RECORD_REJECTED,
+                "action" );
+            return;
+        }
         yaz_log(YLOG_DEBUG, "basename = %s", rr.basenames[0] );
         yaz_log(YLOG_DEBUG, "Operation = %s", rr.operation );
-       if ( !strcmp( rr.operation, "delete" ) ){
-            if ( !srw_req->recordId ){
-                yaz_add_srw_diagnostic(assoc->encode, &srw_res->diagnostics,
-                                       &srw_res->num_diagnostics,
-                                       7, "recordId" );
-            }
-            else {
-                rr.record_id = srw_req->recordId;
-            }
-            if (  !srw_req->recordVersion ){
-                yaz_add_srw_diagnostic(assoc->encode, &srw_res->diagnostics,
-                                       &srw_res->num_diagnostics,
-                                       7, "recordVersion" );
-            }
-            else {
-                rr.record_version = odr_strdup( assoc->encode,
-                                                srw_req->recordVersion );
-                
+       if (!strcmp( rr.operation, "delete"))
+        {
+            if (srw_req->record && !srw_req->record->recordSchema)
+            {
+                rr.record_schema = odr_strdup(
+                    assoc->encode,
+                    srw_req->record->recordSchema);
             }
-            if ( srw_req->recordOldVersion ){
-                rr.record_old_version = odr_strdup(assoc->encode,
-                                                   srw_req->recordOldVersion );
+            if (srw_req->record)
+            {
+                rr.record_data = odr_strdupn(
+                    assoc->encode, 
+                    srw_req->record->recordData_buf,
+                    srw_req->record->recordData_len );
             }
-            if ( srw_req->extraRequestData ){
-                rr.extra_request_data = odr_strdup(assoc->encode,
-                                                   srw_req->extraRequestData );
+            if (extra && extra->extraRecordData_len)
+            {
+                rr.extra_record_data = odr_strdupn(
+                    assoc->encode, 
+                    extra->extraRecordData_buf,
+                    extra->extraRecordData_len );
             }
+            if (srw_req->recordId)
+                rr.record_id = srw_req->recordId;
+            else if (extra && extra->recordIdentifier)
+                rr.record_id = extra->recordIdentifier;
        }
-       else if ( !strcmp( rr.operation, "replace" ) ){
-            if ( !srw_req->recordId ){
-                yaz_add_srw_diagnostic(assoc->encode, &srw_res->diagnostics,
-                                       &srw_res->num_diagnostics,
-                                       7, "recordId" );
-            }
-            else {
+       else if (!strcmp(rr.operation, "replace"))
+        {
+            if (srw_req->recordId)
                 rr.record_id = srw_req->recordId;
+            else if (extra && extra->recordIdentifier)
+                rr.record_id = extra->recordIdentifier;
+            else 
+            {
+                yaz_add_sru_update_diagnostic(
+                    assoc->encode, &srw_res->diagnostics,
+                    &srw_res->num_diagnostics,
+                    YAZ_SRU_UPDATE_MISSING_MANDATORY_ELEMENT_RECORD_REJECTED,
+                    "recordIdentifier");
             }
-            if ( srw_req->record.recordSchema == 0 ){
-                yaz_add_srw_diagnostic(assoc->encode, &srw_res->diagnostics,
-                                       &srw_res->num_diagnostics,
-                                       7, "recordSchema" );
-            }
-            else {
-                rr.record_schema = odr_strdup(assoc->encode,
-                                              srw_req->record.recordSchema );
-            }
-            switch (srw_req->record.recordPacking)
+            if (!srw_req->record)
             {
-            case Z_SRW_recordPacking_string: 
-                rr.record_packing = "string";
-                break;
-            case Z_SRW_recordPacking_XML: 
-                rr.record_packing = "xml";
-                break;
-            case Z_SRW_recordPacking_URL: 
-                rr.record_packing = "url";
-                break;
+                yaz_add_sru_update_diagnostic(
+                    assoc->encode, &srw_res->diagnostics,
+                    &srw_res->num_diagnostics,
+                    YAZ_SRU_UPDATE_MISSING_MANDATORY_ELEMENT_RECORD_REJECTED,
+                    "record");
             }
-            if ( srw_req->record.recordData_len ){
-                rr.record_data = odr_strdupn(assoc->encode, 
-                                             srw_req->record.recordData_buf,
-                                             srw_req->record.recordData_len );
-                rr.request_extra_record = srw_req->extra_record;
+            else 
+            {
+                if (srw_req->record->recordSchema)
+                    rr.record_schema = odr_strdup(
+                        assoc->encode, srw_req->record->recordSchema);
+                if (srw_req->record->recordData_len )
+                {
+                    rr.record_data = odr_strdupn(assoc->encode, 
+                                                 srw_req->record->recordData_buf,
+                                                 srw_req->record->recordData_len );
+                }
+                else 
+                {
+                    yaz_add_sru_update_diagnostic(
+                        assoc->encode, &srw_res->diagnostics,
+                        &srw_res->num_diagnostics,
+                        YAZ_SRU_UPDATE_MISSING_MANDATORY_ELEMENT_RECORD_REJECTED,                                              
+                        "recordData" );
+                }
             }
-            else {
-                yaz_add_srw_diagnostic(assoc->encode, &srw_res->diagnostics,
-                                       &srw_res->num_diagnostics,
-                                       7, "recordData" );
+            if (extra && extra->extraRecordData_len)
+            {
+                rr.extra_record_data = odr_strdupn(
+                    assoc->encode, 
+                    extra->extraRecordData_buf,
+                    extra->extraRecordData_len );
             }
-            if (srw_req->extraRequestData)
-                rr.extra_request_data = odr_strdup(assoc->encode,
-                                                   srw_req->extraRequestData );
        }
-       else if ( !strcmp( rr.operation, "insert" ) )
+       else if (!strcmp(rr.operation, "insert"))
         {
-            rr.record_id = srw_req->recordId; 
-            if ( srw_req->record.recordSchema == 0 ){
-                yaz_add_srw_diagnostic(assoc->encode, &srw_res->diagnostics,
-                                       &srw_res->num_diagnostics,
-                                       7, "recordSchema" );
-            }
-            else {
-                rr.record_schema = odr_strdup(assoc->encode,
-                                              srw_req->record.recordSchema);
-            }
-            switch (srw_req->record.recordPacking)
+            if (srw_req->recordId)
+                rr.record_id = srw_req->recordId; 
+            else if (extra)
+                rr.record_id = extra->recordIdentifier;
+            
+            if (srw_req->record)
             {
-            case Z_SRW_recordPacking_string: 
-                rr.record_packing = "string";
-                break;
-            case Z_SRW_recordPacking_XML: 
-                rr.record_packing = "xml";
-                break;
-            case Z_SRW_recordPacking_URL: 
-                rr.record_packing = "url";
-                break;
-            }
+                if (srw_req->record->recordSchema)
+                    rr.record_schema = odr_strdup(
+                        assoc->encode, srw_req->record->recordSchema);
             
-            if (srw_req->record.recordData_len)
+                if (srw_req->record->recordData_len)
+                    rr.record_data = odr_strdupn(
+                        assoc->encode, 
+                        srw_req->record->recordData_buf,
+                        srw_req->record->recordData_len );
+            }
+            if (extra && extra->extraRecordData_len)
             {
-                rr.record_data = odr_strdupn(assoc->encode, 
-                                             srw_req->record.recordData_buf,
-                                             srw_req->record.recordData_len );
-                rr.request_extra_record = srw_req->extra_record;
+                rr.extra_record_data = odr_strdupn(
+                    assoc->encode, 
+                    extra->extraRecordData_buf,
+                    extra->extraRecordData_len );
             }
-            else
-                yaz_add_srw_diagnostic(assoc->encode, &srw_res->diagnostics,
-                                       &srw_res->num_diagnostics,
-                                       7, "recordData" );
-            if ( srw_req->extraRequestData )
-                rr.extra_request_data = odr_strdup(assoc->encode,
-                                                   srw_req->extraRequestData );
        }
+       else 
+            yaz_add_sru_update_diagnostic(assoc->encode, &srw_res->diagnostics,
+                                          &srw_res->num_diagnostics,
+                                          YAZ_SRU_UPDATE_INVALID_ACTION,
+                                          rr.operation );
+
+        if (srw_req->record)
+        {
+            const char *pack_str = 
+                yaz_srw_pack_to_str(srw_req->record->recordPacking);
+            if (pack_str)
+                rr.record_packing = odr_strdup(assoc->encode, pack_str);
+        }
+
+        if (srw_req->num_recordVersions)
+        {
+            rr.record_versions = srw_req->recordVersions;
+            rr.num_versions = srw_req->num_recordVersions;
+        }
+        if (srw_req->extraRequestData_len)
+        {
+            rr.extra_request_data = odr_strdupn(assoc->encode,
+                                                srw_req->extraRequestData_buf,
+                                                srw_req->extraRequestData_len );
+        }
         if (srw_res->num_diagnostics == 0)
         {
             if ( assoc->init->bend_srw_update)
                 (*assoc->init->bend_srw_update)(assoc->backend, &rr);
-            else {
-                yaz_log( YLOG_WARN, "Got No Update function!");
-                return;
-            }
+            else 
+                yaz_add_sru_update_diagnostic(
+                    assoc->encode, &srw_res->diagnostics,
+                    &srw_res->num_diagnostics,
+                    YAZ_SRU_UPDATE_UNSPECIFIED_DATABASE_ERROR,
+                    "No Update backend handler");
         }
 
         if (rr.uri)
@@ -1616,22 +1623,40 @@ static void srw_bend_update(association *assoc, request *req,
                                        rr.details);
        srw_res->recordId = rr.record_id;
        srw_res->operationStatus = rr.operation_status;
-       srw_res->recordVersion = rr.record_version;
-       srw_res->recordChecksum = rr.record_checksum;
-       srw_res->extraResponseData = rr.extra_response_data;
-        srw_res->record.recordPosition = 0;
+       srw_res->recordVersions = rr.record_versions;
+       srw_res->num_recordVersions = rr.num_versions;
+        if (srw_res->extraResponseData_len)
+        {
+            srw_res->extraResponseData_buf = rr.extra_response_data;
+            srw_res->extraResponseData_len = strlen(rr.extra_response_data);
+        }
        if (srw_res->num_diagnostics == 0 && rr.record_data)
-       {
-            srw_res->record.recordSchema = rr.record_schema;
-            srw_res->record.recordPacking = srw_req->record.recordPacking;
-            srw_res->record.recordData_buf = rr.record_data;
-            srw_res->record.recordData_len = strlen(rr.record_data);
-            srw_res->extra_record = rr.response_extra_record;
-                
-       }
-       else
-            srw_res->record.recordData_len = 0;
-       *http_code = 200;
+        {
+            srw_res->record = yaz_srw_get_record(assoc->encode);
+            srw_res->record->recordSchema = rr.record_schema;
+            if (rr.record_packing)
+            {
+                int pack = yaz_srw_str_to_pack(rr.record_packing);
+
+                if (pack == -1)
+                {
+                    pack = Z_SRW_recordPacking_string;
+                    yaz_log(YLOG_WARN, "Back packing %s from backend",
+                            rr.record_packing);
+                }
+                srw_res->record->recordPacking = pack;
+            }
+            srw_res->record->recordData_buf = rr.record_data;
+            srw_res->record->recordData_len = strlen(rr.record_data);
+            if (rr.extra_record_data)
+            {
+                Z_SRW_extra_record *ex = 
+                    yaz_srw_get_extra_record(assoc->encode);
+                srw_res->extra_record = ex;
+                ex->extraRecordData_buf = rr.extra_record_data;
+                ex->extraRecordData_len = strlen(rr.extra_record_data);
+            }
+        }
     }
 }
 
@@ -1830,12 +1855,9 @@ static void process_http_request(association *assoc, request *req)
         {
             static Z_SOAP_Handler soap_handlers[4] = {
 #if YAZ_HAVE_XML2
-                {"http://www.loc.gov/zing/srw/", 0,
-                 (Z_SOAP_fun) yaz_srw_codec},
-                {"http://www.loc.gov/zing/srw/v1.0/", 0,
-                 (Z_SOAP_fun) yaz_srw_codec},
-                {"http://www.loc.gov/zing/srw/update/", 0,
-                 (Z_SOAP_fun) yaz_ucp_codec},
+                {YAZ_XMLNS_SRU_v1_1, 0, (Z_SOAP_fun) yaz_srw_codec},
+                {YAZ_XMLNS_SRU_v1_0, 0, (Z_SOAP_fun) yaz_srw_codec},
+                {YAZ_XMLNS_UPDATE_v0_9, 0, (Z_SOAP_fun) yaz_ucp_codec},
 #endif
                 {0, 0, 0}
             };
@@ -2107,13 +2129,17 @@ static int process_gdu_response(association *assoc, request *req, Z_GDU *res)
     iochan_setflag(assoc->client_chan, EVENT_OUTPUT);
     assoc->cs_put_mask = EVENT_OUTPUT;
     /* Is there more work to be done? give that to the input handler too */
-#if 1
-    if (request_head(&assoc->incoming))
+    for (;;)
     {
-        yaz_log (YLOG_DEBUG, "more work to be done");
-        iochan_setevent(assoc->client_chan, EVENT_WORK);
+        req = request_head(&assoc->incoming);
+        if (req && req->state == REQUEST_IDLE)
+        {
+            request_deq(&assoc->incoming);
+            process_gdu_request(assoc, req);
+        }
+        else
+            break;
     }
-#endif
     return 0;
 }
 
@@ -2307,13 +2333,12 @@ static Z_APDU *process_initRequest(association *assoc, request *reqb)
     }
 
     yaz_log(log_requestdetail, "Negotiated to v%d: %s", assoc->version, options);
-    assoc->maximumRecordSize = *req->maximumRecordSize;
 
-    if (cb && assoc->maximumRecordSize > cb->maxrecordsize)
-        assoc->maximumRecordSize = cb->maxrecordsize;
-    assoc->preferredMessageSize = *req->preferredMessageSize;
-    if (assoc->preferredMessageSize > assoc->maximumRecordSize)
-        assoc->preferredMessageSize = assoc->maximumRecordSize;
+    if (*req->maximumRecordSize < assoc->maximumRecordSize)
+        assoc->maximumRecordSize = *req->maximumRecordSize;
+
+    if (*req->preferredMessageSize < assoc->preferredMessageSize)
+        assoc->preferredMessageSize = *req->preferredMessageSize;
 
     resp->preferredMessageSize = &assoc->preferredMessageSize;
     resp->maximumRecordSize = &assoc->maximumRecordSize;
@@ -2326,7 +2351,7 @@ static Z_APDU *process_initRequest(association *assoc, request *reqb)
                 assoc->init->implementation_name,
                 odr_prepend(assoc->encode, "GFS", resp->implementationName));
 
-    version = odr_strdup(assoc->encode, "$Revision: 1.96 $");
+    version = odr_strdup(assoc->encode, "$Revision: 1.106 $");
     if (strlen(version) > 10)   /* check for unexpanded CVS strings */
         version[strlen(version)-2] = '\0';
     resp->implementationVersion = odr_prepend(assoc->encode,
@@ -3502,6 +3527,15 @@ static Z_APDU *process_ESRequest(association *assoc, request *reqb, int *fd)
     return apdu;
 }
 
+int bend_assoc_is_alive(bend_association assoc)
+{
+    if (assoc->state == ASSOC_DEAD)
+        return 0; /* already marked as dead. Don't check I/O chan anymore */
+
+    return iochan_is_alive(assoc->client_chan);
+}
+
+
 /*
  * Local variables:
  * c-basic-offset: 4