[Libreoffice-commits] core.git: ucb/source

[Giuseppe Castagno]

 ucb/source/ucp/webdav-neon/DAVResourceAccess.cxx |    4 ++++
 1 file changed, 4 insertions(+)

New commits:
commit 6dcd231892d80d2f130d1b9ebb9cf7dfa1115df1
Author: Giuseppe Castagno <giuseppe.castagno at acca-esse.eu>
Date:   Sat Jul 23 12:55:20 2016 +0200

    tdf#101094 (6) OPTIONS: Do not retry on HTTP error 501
    
    HTTP error 501 the server tells us the used method is not
    implemented, non need to retry.
    
    Change-Id: I4529d2bc27a87e8f3463ded578c6c55e0701ae17
    Reviewed-on: https://gerrit.libreoffice.org/27636
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Giuseppe Castagno <giuseppe.castagno at acca-esse.eu>

diff --git a/ucb/source/ucp/webdav-neon/DAVResourceAccess.cxx b/ucb/source/ucp/webdav-neon/DAVResourceAccess.cxx
index e1ff5d6..51feef6 100644
--- a/ucb/source/ucp/webdav-neon/DAVResourceAccess.cxx
+++ b/ucb/source/ucp/webdav-neon/DAVResourceAccess.cxx
@@ -1197,7 +1197,11 @@ bool DAVResourceAccess::handleException( const DAVException & e, int errorCount
     // if we have a bad connection try again. Up to three times.
     case DAVException::DAV_HTTP_ERROR:
         // retry up to three times, if not a client-side error.
+        // exception: error 501, server side error that
+        // tells us the used method is not implemented
+        // on the server, it's nonsense to insist...
         if ( ( e.getStatus() < 400 || e.getStatus() >= 500 ) &&
+             ( e.getStatus() != 501 ) &&
              errorCount < 3 )
         {
             return true;