SNOW-1650640 changed OCSP fail-open warning message loglevel to warning (from error) (snowflakedb#2045)

sfc-gh-dszmolka · web-flow · commit 66c99bc157ca · 2024-09-06T07:03:32.000+02:00
diff --git a/DESCRIPTION.md b/DESCRIPTION.md
@@ -11,6 +11,7 @@ Source code is also available at: https://github.com/snowflakedb/snowflake-conne
 - v3.12.2 (TBD)
   - Improved implementation of `snowflake.connector.util_text.random_string` to avoid collisions.
   - If the account specifies a region, and that region is in China, the TLD is now inferred to be snowflakecomputing.cn.
+  - Changed loglevel to WARNING for OCSP fail-open warning messages (was: ERROR)
 
 - v3.12.1(August 20,2024)
   - Fixed a bug that logged the session token when renewing a session.
diff --git a/src/snowflake/connector/ocsp_snowflake.py b/src/snowflake/connector/ocsp_snowflake.py
@@ -1076,7 +1076,7 @@ def print_fail_open_warning(ocsp_log) -> None:
             "responder. Details:"
         )
         ocsp_warning = f"{static_warning} \n {ocsp_log}"
-        logger.error(ocsp_warning)
+        logger.warning(ocsp_warning)
 
     def validate_by_direct_connection(
         self,

Original file line number	Diff line number	Diff line change
`@@ -1076,7 +1076,7 @@ def print_fail_open_warning(ocsp_log) -> None:`
`1076`	`1076`	`"responder. Details:"`
`1077`	`1077`	`)`
`1078`	`1078`	`ocsp_warning = f"{static_warning} \n {ocsp_log}"`
`1079`		`- logger.error(ocsp_warning)`
	`1079`	`+ logger.warning(ocsp_warning)`
`1080`	`1080`
`1081`	`1081`	`def validate_by_direct_connection(`
`1082`	`1082`	`self,`