singer-io / tap-marketo

GNU Affero General Public License v3.0
9 stars 17 forks source link

Check existence of export file with request before resuming sync #62

Closed dmosorast closed 4 years ago

dmosorast commented 4 years ago

Description of change

It appears that there's a situation where Marketo marks an export as "Completed", but returns a 404 when requesting the file itself. This is an error scenario as far as the tap is concerned.

This PR adjusts the tap's behavior when resuming to check for a 404 when requesting a 0 byte range of the file, and reporting it to the sync code as not existing if that check returns a 404.

Manual QA steps

Risks

Rollback steps