Skip to content

Conversation

@gsoltis
Copy link
Contributor

@gsoltis gsoltis commented Apr 5, 2021

  • Include the path in the context for uri functions. This allows use of things like org and account ids
  • Include account id in made-up URIs. Some resources don't have published arns, and may conflict between accounts in the same organization (and hence same provider_account_id) if the URIs don't also include the account id.
    • config recorder
    • cloudwatch metric

Fixes #18

@gsoltis gsoltis merged commit 29dca1d into main Apr 5, 2021
@gsoltis gsoltis deleted the add_account_id_uri branch April 5, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

psycopg2.errors.CardinalityViolation: ON CONFLICT DO UPDATE command cannot affect row a second time

1 participant