-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
[Bug]: 408 Timeout during initial import after 175 records #405
Comments
just to make sure I understand the issue, when Fasten attempts to refresh the access token, the refresh fails? |
It appears that way based on the log output. Here's a snippet of the logs surrounding where they start to fail:
All resources above and below of this point continued their respective patterns. In total the first 175 resources completed successfully while the next 167 (the rest) errored out as shown above. At the end of the sync there is an error message that appears to be unrelated. Happy to break that out into it's own issue if it isn't. Putting it below in case it is:
Note: |
Hmm, so every request after the first 175 fails while attempting to refresh the offline token.. that's definitely not how this should work. Let me see if I can replicate this locally. I haven't seen any issues like this in the eClinicalWorks/Healow Sandbox, so I might have to find a way to create a real patient portal account with one of these practices :( |
If you need a guinea pig to play around with some stuff I'm happy to help to the extent I can |
@AnalogJ just wanted to ping this. Still looking for help with this? |
Hey @babbitt Apologies, I've been a bit distracted working on the Patient Summary Export/PDF generation. Can you throw a meeting on my calendar? We can do a live debugging session and maybe I can figure out how to replicate this issue locally. |
hey @babbitt can you pull the latest docker image and see if theres any additional errors being logged during the token refresh? Here's the change I made: fastenhealth/fasten-sources@ba30d17 The log message should be prefixed with |
@AnalogJ
|
ugh. well thats not very helpful, let me see if I can find some other logging I can add. |
Contact Details
[email protected]
What happened?
During initial resource import after connection (Using eClinicalWorks in this case, can check with another type of connection and get back to you) The first 175 resources import fine then the logs start returning 408's and fasten keeps trying to generate a new access token.
Each resource then takes exactly 10s to fail, one after the other.
Based on the fact that 175 seems like to round of a number to be a coincidence; seems like maybe eClinicalWorks is rate limiting you (without knowing anything about either system). Maybe it's worth putting in a watchdog that pauses resource import for X minutes after Y fails (or worse yet hard-coding 175 resource loads per chunk).
Again, huge fan of what you're doing and happy hunt down bugs :)
Flavor
Docker
Version
main#f21ff8b
Relevant log output
The text was updated successfully, but these errors were encountered: