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

Make rate limiting correlate optional #281

Merged

Commits on Mar 13, 2019

  1. Make rate limiting correlate optional

    While looking into an issue in the Terraform provider, I've found that
    the `correlate` values for rate limiting are always being sent
    regardless of whether the value is set or not. This isn't really a big
    deal as I'm assuming the API endpoint just disregard it but for the
    Terraform provider, it does make the state management more difficult and
    messy than what it needs to be due to the nested types.
    
    Instead of always sending the value, I've updated the struct to allow
    `nil` and to omit when the values are empty resulting in a fix for the
    issues downstream.
    jacobbednarz committed Mar 13, 2019
    Copy the full SHA
    fde2244 View commit details
    Browse the repository at this point in the history