Skip to content
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.

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

Experiencing 401 Bad Credentials when credentials are valid #243

Open
dlouseiro opened this issue Jan 17, 2024 · 1 comment
Open

Experiencing 401 Bad Credentials when credentials are valid #243

dlouseiro opened this issue Jan 17, 2024 · 1 comment

Comments

@dlouseiro
Copy link
Contributor

dlouseiro commented Jan 17, 2024

While synchronising data for our github organisation, we've been experiencing consistent HTTP 401 errors with the reason "Bad credentials" which should occur when an invalid token is provided.

Although, this happens after a lot of time into the synchronisation, after performing multiple successful API calls with the same credentials. According to this line of code a similar behaviour has been experienced (random 401 errors - triggering a retry), but not with the reason "Bad Credentials".

All failures experienced were while fetching team members stream, but always in different teams/members. Not consistently on the exact same call.

While trying a curl call to this endpoint using the same token and endpoint call that cause the failure, we receive a 200 OK.

Yet another detail which makes the behaviour even weirder is that if I do an execution with --select teammembers.*, the execution is successful without any failure. With the exact same token.

While an easy solution could be to changethis line of code to retry on any 401 call (not excluding bad credentials), this would cause the tap to retry, even when a bad token is effectively provided.

So, the only solution I see would be to retry on every 401 call, as long as an X amount of API calls was already successfully performed with the same credentials.

@edgarrmondragon
Copy link
Member

@dlouseiro Can you confirm if this is still an issue in the latest version of the tap? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants