SNOW-878067: Retry Strategy (#803)

### Description Regarding issue 575 The PR adds a new retry strategy for login requests with a new base time and adding jitter. It also adds the id and name to the header for login requests ### Checklist - [x] Code compiles correctly - [x] Code is formatted according to [Coding Conventions](../CodingConventions.md) - [x] Created tests which fail without the change (if possible) - [x] All tests passing (`dotnet test`) - [ ] Extended the README / documentation, if necessary - [x] Provide JIRA issue id (if possible) or GitHub issue id in PR name
snowflakedb · Nov 6, 2023 · 7f73791 · 7f73791
1 parent eb2e0ab
commit 7f73791
Show file tree

Hide file tree

Showing 12 changed files with 537 additions and 122 deletions.
diff --git a/README.md b/README.md
@@ -144,7 +144,8 @@ The following table lists all valid connection properties:
 | SCHEMA                         | No       |                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                 |
 | USER                           | Depends  | If AUTHENTICATOR is set to `externalbrowser` this is optional. For native SSO through Okta, set this to the login name for your identity provider (IdP).                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        |
 | WAREHOUSE                      | No       |                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                 |
-| CONNECTION_TIMEOUT             | No       | Total timeout in seconds when connecting to Snowflake. The default is 120 seconds                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                               |
+| CONNECTION_TIMEOUT             | No       | Total timeout in seconds when connecting to Snowflake. The default is 300 seconds                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                               |
+| RETRY_TIMEOUT                  | No       | Total timeout in seconds for supported endpoints of retry policy. The default is 300 seconds. The value can only be increased from the default value or set to 0 for infinite timeout                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                               |
 | MAXHTTPRETRIES                 | No       | Maximum number of times to retry failed HTTP requests (default: 7). You can set `MAXHTTPRETRIES=0` to remove the retry limit, but doing so runs the risk of the .NET driver infinitely retrying failed HTTP calls.                                                                                                                                                                                                                                                                                                                                                                                                                                                                                              |
 | CLIENT_SESSION_KEEP_ALIVE      | No       | Whether to keep the current session active after a period of inactivity, or to force the user to login again. If the value is `true`, Snowflake keeps the session active indefinitely, even if there is no activity from the user. If the value is `false`, the user must log in again after four hours of inactivity. The default is `false`. Setting this value overrides the server session property for the current session.                                                                                                                                                                                                                                                                                |
 | DISABLERETRY                   | No       | Set this property to `true` to prevent the driver from reconnecting automatically when the connection fails or drops. The default value is `false`.                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                             |