Prevent an infinite loop

We have confirmed, that sometimes the exception handling returns a null client wrapper. When this circumstance happens, and it has several times for us, the code goes into an infinite loop because the wrapper remains null and is never set again. Moving the retry attempts increment before the retry (and outside of the exceptions) prevents this infinite loop.  Executing the Query as normal when a wrapper is unavailable also ensures a retry is made when the wrapper is null.

Author: wildcarrotsafari

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md

@@ -202,12 +202,21 @@ public static void ExecuteQueryWithIncrementalRetry(this ClientContext clientCon
             }
             else
             {
-                // retry the previous request
+                //increment the retry count                
+                retryAttempts++;
+                
+                // retry the previous request using wrapper
                 if (wrapper != null && wrapper.Value != null)
                 {
                     clientContext.RetryQuery(wrapper.Value);
                     return;
-                }
+                } 
+                // retry the previous request as normal
+                else
+                {
+                    clientContext.ExecuteQuery();
+                    return;
+                }                
             }
         }
         catch (WebException ex)
@@ -237,8 +246,7 @@ public static void ExecuteQueryWithIncrementalRetry(this ClientContext clientCon
                 // Delay for the requested seconds
                 Thread.Sleep(retryAfterInterval * 1000);
 
-                // Increase counters
-                retryAttempts++;
+                // Increase counters          
                 backoffInterval = backoffInterval * 2;
             }
             else