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

add property to configure rate limit #3

Open
david-alpert-nl opened this issue Jan 10, 2024 · 1 comment
Open

add property to configure rate limit #3

david-alpert-nl opened this issue Jan 10, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@david-alpert-nl
Copy link

We are using this provider to try populating a new Stack with ~35 global fields and content types.

When we run terraform plan and the provider tries to refresh state or apply we are running into many 429 (too many request) responses from the ContentStack Management API.

We would like a method to tune the number of API requests per second to match our ContentStack API Rate Limit and avoid going over.

@david-alpert-nl david-alpert-nl added enhancement New feature or request triage Needs triage labels Jan 10, 2024
@demeyerthom
Copy link
Member

Hi @david-alpert-nl thanks for reaching out!

This would indeed be useful. We could probably implement a rate limiter on the RoundTripper by adding something like https://pkg.go.dev/golang.org/x/time/rate to this.

I will put this on the backlog to look into, but due to priorities it might take a while. Feel free to send me a PR to add this if you need it quickly!

@demeyerthom demeyerthom removed the triage Needs triage label Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants