Skip to content

fix: improve OAuth refresh token rejection error message #1130

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Patrick Nilan (pnilan) merged 2 commits into from
Aug 25, 2026
Merged

fix: improve OAuth refresh token rejection error message #1130

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b9efbe6
fix: improve OAuth refresh token rejection error message
devin-ai-integration[bot] Aug 24, 2026
e91bda4
fix: truncate OAuth provider response details
devin-ai-integration[bot] Aug 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 9 additions & 3 deletions airbyte_cdk/sources/streams/http/requests_native_auth/abstract_oauth.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -334,10 +334,16 @@ def _make_handled_request(self) -> Any:
failure_type=FailureType.transient_error,
)
if self._wrap_refresh_token_exception(e):
message = "Refresh token is invalid or expired. Please re-authenticate from Sources/<your source>/Settings."
raise AirbyteTracedException(
internal_message=message, message=message, failure_type=FailureType.config_error
response_info = (
f"HTTP {e.response.status_code}: {e.response.text[:1000]}"
if e.response is not None
else str(e)
)
raise AirbyteTracedException(
internal_message=f"Refresh token rejected by the OAuth token endpoint. {response_info}",
message="Refresh token was rejected by the OAuth provider (invalid, expired, or already used). Re-authenticate this source's credentials in its connection settings.",
failure_type=FailureType.config_error,
) from e
raise
except Exception as e:
raise AirbyteTracedException(
Expand Down
8 changes: 6 additions & 2 deletions unit_tests/sources/streams/http/requests_native_auth/test_requests_native_auth.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -642,8 +642,12 @@ def test_refresh_access_token_wrapped(
oauth.refresh_access_token()

if wrapped:
error_message = "Refresh token is invalid or expired. Please re-authenticate from Sources/<your source>/Settings."
assert exc_info.value.internal_message == error_message
error_message = "Refresh token was rejected by the OAuth provider (invalid, expired, or already used). Re-authenticate this source's credentials in its connection settings."
assert exc_info.value.internal_message.startswith(
"Refresh token rejected by the OAuth token endpoint."
)
assert f"HTTP {response_code}" in exc_info.value.internal_message
assert response_value in exc_info.value.internal_message
assert exc_info.value.message == error_message
assert exc_info.value.failure_type == FailureType.config_error

Expand Down
Loading