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: amazon-seller-partner - list_financial_events: Reduce max days apart to 14 days #55170

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

keiththompson
Copy link
Contributor

What

The documentation claims that days between PostedBefore and PostedAfter should be no more than 180 days. However I have observed this failing very quickly with any value above 14 days with the following error:

{
  "type": "LOG",
  "log": {
    "level": "ERROR",
    "message": "'GET' request to 'https://sellingpartnerapi-eu.amazon.com/finances/v0/financialEvents?MaxResultsPerPage=100&PostedAfter=2024-09-02T22%3A20%3A45Z&PostedBefore=2025-03-01T22%3A15%3A46Z' failed with status code '400' and error message: 'Invalid Input'. Request (body): 'None'. Response (body): '{'errors': [{'code': 'InvalidInput', 'message': 'Invalid Input', 'details': ''}]}'. Response (headers): '{'Server': 'Server', 'Date': 'Sat, 01 Mar 2025 22:20:49 GMT', 'Content-Type': 'application/json', 'Content-Length': '117', 'Connection': 'keep-alive', 'x-amz-rid': 'xxx', 'x-amzn-RateLimit-Limit': '0.5', 'x-amzn-RequestId': 'xxx', 'x-amz-apigw-id': 'xxx', 'X-Amzn-Trace-Id': 'xxx', 'x-amzn-ErrorType': 'InvalidInputException', 'Vary': 'Content-Type,Accept-Encoding,User-Agent', 'Strict-Transport-Security': 'max-age=47474747; includeSubDomains; preload'}'."
  }
}

How

Reduced the gap to 14 days

Review guide

  1. airbyte-integrations/connectors/source-amazon-seller-partner/source_amazon_seller_partner/manifest.yaml

User Impact

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Copy link

vercel bot commented Mar 2, 2025

@keiththompson is attempting to deploy a commit to the Airbyte Growth Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

2 participants