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

[source-quickbooks] token expires every 24h #49842

Open
1 task done
berz01 opened this issue Dec 17, 2024 · 3 comments
Open
1 task done

[source-quickbooks] token expires every 24h #49842

berz01 opened this issue Dec 17, 2024 · 3 comments

Comments

@berz01
Copy link

berz01 commented Dec 17, 2024

Connector Name

Quickbooks

Connector Version

QuickBooks v3.0.26

What step the error happened?

None

Relevant information

Strange behavior on disconnects every 24 hours with the QuickBooks connector.

It doesn't appear the token refresh every 24 hours like its supposed to any more? My oauth production apps are totally fine. Is anyone else experiencing only running after the initial 24 hours of a successful test & entering a new refresh token?

Image

Let me know if I can assist with updating the refresh capabilities

Relevant log output

Configuration check failed
'Unauthorized. Please ensure you are authenticated correctly.'

Contribute

  • Yes, I want to contribute
@natikgadzhi
Copy link
Contributor

I see. Must be that the token can't refresh quickly enough. @bnchrch @erohmensing @pabloescoder do any of our new OAuth things put us in a good spot to test them on Quickbooks tomorrow morning, see if we can fix the obvious problems?

@pabloescoder
Copy link
Collaborator

@natikgadzhi @bnchrch @erohmensing I think I know what's wrong.
I read the QB docs, they invalidate the refresh token every time it's used to request a new access token & send a new refresh token.

So we'll have to use the "override config with refresh token response" checkbox in the builder which creates a refresh_token_updater in the yaml.

I'll raise a PR for this.

Screenshot_2024_1218_102113.png

@berz01
Copy link
Author

berz01 commented Dec 20, 2024

thanks guys, I knew something was up and I know how important QuickBooks is to your marketplace and customers. I could tell it wasn't refreshing with the 'newest' refresh token.

Let me know if you need more testing you can dm me directly

@marcosmarxm marcosmarxm changed the title Quickbooks Disconnects [source-quickbooks] token expires every 24h Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants