1
0
mirror of synced 2025-12-20 02:23:30 -05:00

11 Commits

Author SHA1 Message Date
Brian Lai
262018744a fix(source-github): Graceful error handling of invalid credentials when running all sync operations (#67584)
Fixes: https://github.com/airbytehq/airbyte-internal-issues/issues/14576

## What

We had a previous swarm issue that ended up being a customer
misconfiguration due to expired or invalid creds. That being said, the
error message was particularly unhelpful w/ a bunch of python stack
traces. This replace that with far more useful information:

```
{"type":"CONNECTION_STATUS","connectionStatus":{"status":"FAILED","message":"'Github credentials have expired or changed, please review your credentials and re-authenticate or renew your access token.'"}}
``` 

## How

The main thing to call out here is that we want to get in the habit of
using our dedicated `HttpClient` which gives us better error handling
and just default behavior as a whole instead of using the `requests`
library. Otherwise the rest is pretty straightforward

## Review guide

nah

## User Impact

none

## Can this PR be safely reverted and rolled back?

- [x] YES 💚
- [ ] NO 
2025-10-13 16:13:18 -07:00
Daryna Ishchenko
7e5c616453 fix(source-github): fix min time to wait on token rate limits (#67589)
## What
there is an issue while processing min time to wait on token rate
limits:
.seconds - returns positive value 
fixed to
.total_seconds() - returns both negative and positive values. (negative
when reset time for token is older then datetime now)

## How
<!--
* Describe how code changes achieve the solution.
-->

## Review guide
<!--
1. `x.py`
2. `y.py`
-->

## User Impact
<!--
* What is the end result perceived by the user?
* If there are negative side effects, please list them. 
-->

## Can this PR be safely reverted and rolled back?
<!--
* If unsure, leave it blank.
-->
- [ ] YES 💚
- [ ] NO 
2025-10-09 16:29:34 +03:00
Daryna Ishchenko
8bfaa23b2f chore(source-github) update to airbyte cdk v7 (#66736) 2025-10-02 13:29:00 +03:00
Aaron ("AJ") Steers
83ecbe0fc3 CI: apply pre-commit format fix from #49806 (#49852) 2024-12-18 14:05:43 -08:00
Artem Inzhyyants
66cd528340 🐛 Source GitHub: Fix MultipleTokenAuthenticator sleep time (#34598) 2024-01-29 11:29:32 +01:00
Artem Inzhyyants
53851609f8 🐛 Source GitHub: Fix MultipleToken rotation logic (#34503) 2024-01-26 15:15:54 +01:00
Serhii Chvaliuk
30d8bb4133 Source Github: implement client-side throttling of requests (#25793)
Signed-off-by: Serhii Chvaliuk <grubberr@gmail.com>
2023-05-17 22:18:37 +03:00
Serhii Chvaliuk
5053b278f5 Source Github: fix caching for get_starting_point in stream "Commits" (#24398)
Signed-off-by: Sergey Chvalyuk <grubberr@gmail.com>
2023-03-27 13:13:01 +03:00
Cole Snodgrass
2e099acc52 update headers from 2022 -> 2023 (#22594)
* It's 2023!

* 2022 -> 2023

---------

Co-authored-by: evantahler <evan@airbyte.io>
2023-02-08 13:01:16 -08:00
Serhii Chvaliuk
255a5bbd5e 🐛 Source Github: validate input organizations and repositories (#15730)
Signed-off-by: Sergey Chvalyuk <grubberr@gmail.com>
2022-08-20 00:48:33 +03:00
Serhii Chvaliuk
c10a268bde 🐛 Source Github: add incremental for WorkflowRuns (#13115)
* add incremental for `workflowruns`

Signed-off-by: Sergey Chvalyuk <grubberr@gmail.com>
2022-05-27 21:01:28 +03:00