serf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kot...@apache.org
Subject svn commit: r1805341 - /serf/branches/1.3.x/STATUS
Date Thu, 17 Aug 2017 23:03:37 GMT
Author: kotkov
Date: Thu Aug 17 23:03:36 2017
New Revision: 1805341

URL: http://svn.apache.org/viewvc?rev=1805341&view=rev
Log:
* STATUS: Nominate r1804534, r1804543 and r1804553 (Fix error handling
  when reading the outgoing request's body).

Modified:
    serf/branches/1.3.x/STATUS

Modified: serf/branches/1.3.x/STATUS
URL: http://svn.apache.org/viewvc/serf/branches/1.3.x/STATUS?rev=1805341&r1=1805340&r2=1805341&view=diff
==============================================================================
--- serf/branches/1.3.x/STATUS (original)
+++ serf/branches/1.3.x/STATUS Thu Aug 17 23:03:36 2017
@@ -19,6 +19,25 @@ Candidate changes:
    Votes:
      +1: stsp, astieger
 
+ * r1804534, r1804543, r1804553
+   Fix error handling when reading the outgoing request's body by not
+   ignoring such errors in some cases.
+   Justification:
+     Ignoring errors can cause invalid or undefined behavior further
+     down the line.
+   Branch:
+     ^/serf/branches/1.3.x-fix-outgoing-request-err
+   Notes:
+     - On trunk, this issue has been fixed in the process of adding
+       the pump.c layer.
+     - An example of a condition where the error could be incorrectly ignored
+       is Subversion's commit, if the error occurs when reading the body of
+       an outgoing PUT request.
+     - The added test doesn't fail without the fix, but is merely added to
+       cover the related code.  (See the r1804541 log message for details.)
+   Votes:
+     +1: kotkov
+
 Veto-blocked changes:
 =====================
 



Mime
View raw message