Skip to content
This repository has been archived by the owner on Nov 30, 2022. It is now read-only.

Update model to support rate limitis #1292

Closed
earmenda opened this issue Sep 12, 2022 · 1 comment
Closed

Update model to support rate limitis #1292

earmenda opened this issue Sep 12, 2022 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@earmenda
Copy link
Contributor

earmenda commented Sep 12, 2022

Is your feature request related to a specific problem?

We need to give users a way to set rate limits for calls within their connectors.

Describe the solution you'd like

A rate_limit config should exist within connectors or saas connectors config:

    "rate_limit_config" : [
      {
          "rate" : 100,
          "period" : seconds,
          "custom_key" : my_custom_key
      },
      {
          "rate" : 500000,
          "period" : days,
          "custom_key" : my_custom_key2
      }
    ]

Additional context

We need to explore if it is feasible to apply rate limits to all connectors or if it just makes sense within saas connectors. We will prioritize saas connectors for this feature if it makes sense to.

@earmenda
Copy link
Contributor Author

earmenda commented Oct 4, 2022

Closing in favor of ethyca/fides#1221

@earmenda earmenda closed this as completed Oct 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant