Change raise_for_status default to False

dlt-hub · Mar 4, 2024 · e40579b · e40579b
1 parent fc34dd0
commit e40579b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/dlt/sources/helpers/requests/retry.py b/dlt/sources/helpers/requests/retry.py
@@ -177,7 +177,7 @@ def __init__(
             Union[TimedeltaSeconds, Tuple[TimedeltaSeconds, TimedeltaSeconds]]
         ] = DEFAULT_TIMEOUT,
         max_connections: int = 50,
-        raise_for_status: bool = True,
+        raise_for_status: bool = False,
         status_codes: Sequence[int] = DEFAULT_RETRY_STATUS,
         exceptions: Sequence[Type[Exception]] = DEFAULT_RETRY_EXCEPTIONS,
         request_max_attempts: int = RunConfiguration.request_max_attempts,

diff --git a/dlt/sources/helpers/requests/session.py b/dlt/sources/helpers/requests/session.py
@@ -36,7 +36,7 @@ def __init__(
         timeout: Optional[
             Union[TimedeltaSeconds, Tuple[TimedeltaSeconds, TimedeltaSeconds]]
         ] = DEFAULT_TIMEOUT,
-        raise_for_status: bool = True,
+        raise_for_status: bool = False,
     ) -> None:
         super().__init__()
         self.timeout = _timeout_to_seconds(timeout)