feat(http): change error types that are caught

pafi-code · pafi-code · commit d92597d51e35 · 2025-11-24T14:27:56.000+01:00
diff --git a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/_log_exporter/__init__.py b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/_log_exporter/__init__.py
@@ -199,7 +199,7 @@ def export(
                         resp.text,
                     )
                     return LogExportResult.FAILURE
-            except Exception as error:
+            except requests.exceptions.RequestException as error:
                 _logger.error("Failed to export logs batch reason: %s", error)
 
             if (
diff --git a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/metric_exporter/__init__.py b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/metric_exporter/__init__.py
@@ -244,7 +244,7 @@ def export(
                         resp.text,
                     )
                     return MetricExportResult.FAILURE
-            except Exception as error:
+            except requests.exceptions.RequestException as error:
                 _logger.error(
                     "Failed to export metrics batch reason: %s", error
                 )
diff --git a/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py b/exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py
@@ -192,7 +192,7 @@ def export(self, spans: Sequence[ReadableSpan]) -> SpanExportResult:
                         resp.text,
                     )
                     return SpanExportResult.FAILURE
-            except Exception as error:
+            except requests.exceptions.RequestException as error:
                 _logger.error("Failed to export span batch reason: %s", error)
 
             if (

Original file line number	Diff line number	Diff line change
`@@ -199,7 +199,7 @@ def export(`
`199`	`199`	`resp.text,`
`200`	`200`	`)`
`201`	`201`	`return LogExportResult.FAILURE`
`202`		`- except Exception as error:`
	`202`	`+ except requests.exceptions.RequestException as error:`
`203`	`203`	`_logger.error("Failed to export logs batch reason: %s", error)`
`204`	`204`
`205`	`205`	`if (`
Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ def export(`
`244`	`244`	`resp.text,`
`245`	`245`	`)`
`246`	`246`	`return MetricExportResult.FAILURE`
`247`		`- except Exception as error:`
	`247`	`+ except requests.exceptions.RequestException as error:`
`248`	`248`	`_logger.error(`
`249`	`249`	`"Failed to export metrics batch reason: %s", error`
`250`	`250`	`)`
Original file line number	Diff line number	Diff line change
`@@ -192,7 +192,7 @@ def export(self, spans: Sequence[ReadableSpan]) -> SpanExportResult:`
`192`	`192`	`resp.text,`
`193`	`193`	`)`
`194`	`194`	`return SpanExportResult.FAILURE`
`195`		`- except Exception as error:`
	`195`	`+ except requests.exceptions.RequestException as error:`
`196`	`196`	`_logger.error("Failed to export span batch reason: %s", error)`
`197`	`197`
`198`	`198`	`if (`