Skip to content

Commit 8ff11cf

Browse files
committed
include request_url
1 parent a3df9c5 commit 8ff11cf

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

airbyte_cdk/sources/streams/http/http_client.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ def _send_with_retry(
259259
max_tries = max(0, max_retries) + 1
260260
max_time = self._max_time
261261

262-
user_backoff_handler = user_defined_backoff_handler(max_tries=max_tries, max_time=max_time)(
262+
user_backoff_handler = user_defined_backoff_handler(max_tries=max_tries, max_time=max_time, request_url=request.url)(
263263
self._send
264264
)
265265
rate_limit_backoff_handler = rate_limit_default_backoff_handler(max_tries=max_tries)

airbyte_cdk/sources/streams/http/rate_limiting.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ def should_give_up(exc: Exception) -> bool:
101101

102102

103103
def user_defined_backoff_handler(
104-
max_tries: Optional[int], max_time: Optional[int] = None, **kwargs: Any
104+
max_tries: Optional[int], max_time: Optional[int] = None, request_url: Optional[str] = None, **kwargs: Any
105105
) -> Callable[[SendRequestCallableType], SendRequestCallableType]:
106106
def sleep_on_ratelimit(details: Mapping[str, Any]) -> None:
107107
_, exc, _ = sys.exc_info()
@@ -112,7 +112,7 @@ def sleep_on_ratelimit(details: Mapping[str, Any]) -> None:
112112
)
113113
retry_after = exc.backoff
114114
# include logging og the current time to help with debugging
115-
logger.info(f"Retrying. Sleeping for {retry_after} seconds at {time.strftime('%Y-%m-%d %H:%M:%S')}")
115+
logger.info(f"Retrying. Sleeping for {retry_after} seconds at {time.strftime('%Y-%m-%d %H:%M:%S')} for URL: {request_url or 'unknown'}")
116116
time.sleep(retry_after + 1) # extra second to cover any fractions of second
117117

118118
def log_give_up(details: Mapping[str, Any]) -> None:

0 commit comments

Comments
 (0)