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

update key endpoint changed #2209

Merged
merged 1 commit into from
Apr 15, 2019
Merged

update key endpoint changed #2209

merged 1 commit into from
Apr 15, 2019

Conversation

dencoded
Copy link
Contributor

added changes for https://github.com/TykTechnologies/tyk-analytics/issues/996

The problem was because we were setting quota_renews to 0 when updating key with suppress_reset=1 as it forces starting new quota renewal period (which equal to reseting quotas)

@dencoded dencoded requested a review from buger April 15, 2019 03:53
@buger buger merged commit 32859da into master Apr 15, 2019
@buger buger deleted the update-key-reset-quota-fix branch April 15, 2019 07:31
buger pushed a commit that referenced this pull request Apr 15, 2019
added changes for TykTechnologies/tyk-analytics#996

The problem was because we were setting `quota_renews` to 0 when updating key with `suppress_reset=1` as it forces starting new quota renewal period (which equal to reseting quotas)
buger pushed a commit that referenced this pull request May 4, 2019
added changes for TykTechnologies/tyk-analytics#996

The problem was because we were setting `quota_renews` to 0 when updating key with `suppress_reset=1` as it forces starting new quota renewal period (which equal to reseting quotas)
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.

None yet

2 participants