Hello community,

here is the log from the commit of package libserf for openSUSE:Factory checked 
in at 2014-06-10 14:39:17
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/libserf (Old)
 and      /work/SRC/openSUSE:Factory/.libserf.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "libserf"

Changes:
--------
--- /work/SRC/openSUSE:Factory/libserf/libserf.changes  2014-04-30 
15:07:48.000000000 +0200
+++ /work/SRC/openSUSE:Factory/.libserf.new/libserf.changes     2014-06-10 
14:39:19.000000000 +0200
@@ -1,0 +2,9 @@
+Mon Jun  9 18:07:29 UTC 2014 - [email protected]
+
+- Serf 1.3.6 [bnc#881902]
+  This release fixes an instability introduced in serf 1.3.5
+  * Revert r2319 from serf 1.3.5: this change was making serf call
+    handle_response multiple times in case of an error response,
+    leading to unexpected behavior.
+
+-------------------------------------------------------------------

Old:
----
  serf-1.3.5.tar.bz2

New:
----
  serf-1.3.6.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ libserf.spec ++++++
--- /var/tmp/diff_new_pack.xVM8Ms/_old  2014-06-10 14:39:20.000000000 +0200
+++ /var/tmp/diff_new_pack.xVM8Ms/_new  2014-06-10 14:39:20.000000000 +0200
@@ -23,7 +23,7 @@
 %define minor  3
 %define SHLIBVER %{major}.%{minor}.0
 Name:           libserf
-Version:        1.3.5
+Version:        1.3.6
 Release:        0
 Summary:        High-Performance Asynchronous HTTP Client Library
 License:        Apache-2.0

++++++ serf-1.3.5.tar.bz2 -> serf-1.3.6.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/serf-1.3.5/CHANGES new/serf-1.3.6/CHANGES
--- old/serf-1.3.5/CHANGES      2014-04-27 22:30:56.000000000 +0200
+++ new/serf-1.3.6/CHANGES      2014-06-09 18:05:27.000000000 +0200
@@ -1,3 +1,7 @@
+Serf 1.3.6 [2014-06-09, from /tags/1.3.6, rxxxx]
+  Revert r2319 from serf 1.3.5: this change was making serf call 
handle_response
+    multiple times in case of an error response, leading to unexpected 
behavior.
+
 Serf 1.3.5 [2014-04-27, from /tags/1.3.5, rxxxx]
   Fix issue #125: no reverse lookup during Negotiate authentication for 
proxies.
   Fix a crash caused by incorrect reuse of the ssltunnel CONNECT request 
(r2316)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/serf-1.3.5/auth/auth.c new/serf-1.3.6/auth/auth.c
--- old/serf-1.3.5/auth/auth.c  2014-04-21 16:56:02.000000000 +0200
+++ new/serf-1.3.6/auth/auth.c  2014-06-07 09:43:40.000000000 +0200
@@ -408,7 +408,6 @@
                consider the reponse body as invalid and discard it. */
             status = discard_body(response);
             *consumed_response = 1;
-
             if (!APR_STATUS_IS_EOF(status)) {
                 return status;
             }
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/serf-1.3.5/outgoing.c new/serf-1.3.6/outgoing.c
--- old/serf-1.3.5/outgoing.c   2014-04-21 16:56:02.000000000 +0200
+++ new/serf-1.3.6/outgoing.c   2014-06-07 09:43:40.000000000 +0200
@@ -916,22 +916,21 @@
      * themselves by not registering credential callbacks.
      */
     if (request->conn->ctx->cred_cb) {
-        status = serf__handle_auth_response(&consumed_response,
-                                            request,
-                                            request->resp_bkt,
-                                            request->handler_baton,
-                                            pool);
+      status = serf__handle_auth_response(&consumed_response,
+                                          request,
+                                          request->resp_bkt,
+                                          request->handler_baton,
+                                          pool);
 
-        if (SERF_BUCKET_READ_ERROR(status)) {
-            /* Report the request as 'died'/'cancelled' to the application */
-            (void)(*request->handler)(request,
-                                      NULL,
-                                      request->handler_baton,
-                                      pool);
-        }
+      /* If there was an error reading the response (maybe there wasn't
+         enough data available), don't bother passing the response to the
+         application.
 
-        if (status)
-            return status;
+         If the authentication was tried, but failed, pass the response
+         to the application, maybe it can do better. */
+      if (status) {
+          return status;
+      }
     }
 
     if (!consumed_response) {
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/serf-1.3.5/serf.h new/serf-1.3.6/serf.h
--- old/serf-1.3.5/serf.h       2014-04-02 20:13:22.000000000 +0200
+++ new/serf-1.3.6/serf.h       2014-06-07 09:44:30.000000000 +0200
@@ -1062,7 +1062,7 @@
 /* Version info */
 #define SERF_MAJOR_VERSION 1
 #define SERF_MINOR_VERSION 3
-#define SERF_PATCH_VERSION 5
+#define SERF_PATCH_VERSION 6
 
 /* Version number string */
 #define SERF_VERSION_STRING APR_STRINGIFY(SERF_MAJOR_VERSION) "." \

-- 
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to