-
Notifications
You must be signed in to change notification settings - Fork 22
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
bin/backup_contacts.php dying #18
Comments
Note to self: At a minimum this code should check that the Flickr API hasn't failed before purging contacts. |
This is sort of half-fixed, now. Specifically I've added retry code: I'll leave this ticket open until I tweak the code to buffer all new contacts (read: retries) before purging anything that's already in the database. |
Fixed: c9824f5 |
Been dealing w/ work issues so haven't gotten a chance to take much of a look. Did a merge, here are the results: backup contacts for lhl array ( 'ok' => 0, 'error' => 'Unable to fetch contacts: The Flickr API did not return any contacts', ) |
Grrrrnnnn. The good news is that it appears as though the retry code is working. This makes it seems like there's still a bug though: page: 1/ tries: 1/10 ok: 1 That second line should say "page: 2/4" The bad news is that the Flickr API continues to wig out. It's almost as though contacts are being stored in Vespa or something... |
I'm 95% sure that getContacts is hitting the DBs. There was some flip-flop for a while there due to some shit I don't remember, but I think it should be hitting the DBs. |
Also, just to be clear: This is no longer purging all the contacts for a user before re-fetching them via the API. If there is a "fatal" error re-fetching contacts it just means that the local database is not updated. All previous contacts are left untouched. |
I'm seeing this intermittently as well (as of c9824f5). |
So this happens:
Will debug and see if I can pinpoint at least where it's dying...
The text was updated successfully, but these errors were encountered: