Title: Potential request spamming after initial cooldown surpassed · Issue #137 · Azure/AppConfiguration-JavaScriptProvider · GitHub
Open Graph Title: Potential request spamming after initial cooldown surpassed · Issue #137 · Azure/AppConfiguration-JavaScriptProvider
X Title: Potential request spamming after initial cooldown surpassed · Issue #137 · Azure/AppConfiguration-JavaScriptProvider
Description: Hey Folks. We recently had a situation in one of our environments, where usage of the refresh-api somehow overwhelmed our azconfig instances, which then gave HTTP 429 Too-many-requests, occasionally - especially during peak-load times of...
Open Graph Description: Hey Folks. We recently had a situation in one of our environments, where usage of the refresh-api somehow overwhelmed our azconfig instances, which then gave HTTP 429 Too-many-requests, occasionall...
X Description: Hey Folks. We recently had a situation in one of our environments, where usage of the refresh-api somehow overwhelmed our azconfig instances, which then gave HTTP 429 Too-many-requests, occasionall...
Opengraph URL: https://github.com/Azure/AppConfiguration-JavaScriptProvider/issues/137
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Potential request spamming after initial cooldown surpassed","articleBody":"Hey Folks.\n\nWe recently had a situation in one of our environments, where usage of the `refresh`-api somehow overwhelmed our `azconfig` instances, which then gave HTTP `429 Too-many-requests`, occasionally - especially during peak-load times of our app.\n\nWe used [1.0.0-preview4](https://github.com/Azure/AppConfiguration-JavaScriptProvider/releases/tag/1.0.0-preview.4) at this time and thx to @lorenzadomat found out, that the refresh-interval might be ignored, somehow.\n\nI then took the opportunity to narrow this even further down. Seems like starting with [1.1.0](https://github.com/Azure/AppConfiguration-JavaScriptProvider/releases/tag/1.1.0) / [b1cf8099fbf8737de3d38299be2871d4547c7cac ](https://github.com/Azure/AppConfiguration-JavaScriptProvider/pull/65/commits/b1cf8099fbf8737de3d38299be2871d4547c7cac), this behavior dissappeared.\n\nLooks like prior 1.1.0, sentinel keys got queried every time, after the initial `refreshIntervalInMs` has been surpassed - before this has happened, everything works as expected - that's why this was not that easy to reproduce ;)\n\nJust wanted to let you know about this, especially as this might have been fixed \"by accident\" and not on purpose.\n\nI have a simple repro over at my [repro-repo](https://github.com/earloc/repros/tree/main/src/azure/AppConfiguration-JavaScriptProvider#httpsgithubcomazureappconfiguration-javascriptproviderissues137), which hopefully showcases the problem clear enough.\n\nLet me know if this somehow might help in preventing some future regressions. We will probably notice, at least.\n","author":{"url":"https://github.com/earloc","@type":"Person","name":"earloc"},"datePublished":"2024-11-30T19:00:44.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":6},"url":"https://github.com/137/AppConfiguration-JavaScriptProvider/issues/137"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:2362313b-e6b9-c4ad-a134-893395568e47 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | A1A6:13F3B3:4AC38:640C6:698EDC4F |
| html-safe-nonce | 3956bf474d757564a4d3ab3e22200b1463b084f7d5267a7e8d2f95bc0bdcef0e |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBMUE2OjEzRjNCMzo0QUMzODo2NDBDNjo2OThFREM0RiIsInZpc2l0b3JfaWQiOiIxNjY3Mjc5OTUyMTU4ODQxOTM1IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 0b50f01c35bed04d279995abcf3408655f82a50ad2022b503d36c6db0a8e75da |
| hovercard-subject-tag | issue:2707835148 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/Azure/AppConfiguration-JavaScriptProvider/137/issue_layout |
| twitter:image | https://opengraph.githubassets.com/fc40c1710736a2283cdf3304d17d7933553f9d2ae78e13d10323f7e36fc495ea/Azure/AppConfiguration-JavaScriptProvider/issues/137 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/fc40c1710736a2283cdf3304d17d7933553f9d2ae78e13d10323f7e36fc495ea/Azure/AppConfiguration-JavaScriptProvider/issues/137 |
| og:image:alt | Hey Folks. We recently had a situation in one of our environments, where usage of the refresh-api somehow overwhelmed our azconfig instances, which then gave HTTP 429 Too-many-requests, occasionall... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | earloc |
| hostname | github.com |
| expected-hostname | github.com |
| None | cb2828a801ee6b7be618f3ac76fbf55def35bbc30f053a9c41bf90210b8b72ba |
| turbo-cache-control | no-preview |
| go-import | github.com/Azure/AppConfiguration-JavaScriptProvider git https://github.com/Azure/AppConfiguration-JavaScriptProvider.git |
| octolytics-dimension-user_id | 6844498 |
| octolytics-dimension-user_login | Azure |
| octolytics-dimension-repository_id | 688057518 |
| octolytics-dimension-repository_nwo | Azure/AppConfiguration-JavaScriptProvider |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 688057518 |
| octolytics-dimension-repository_network_root_nwo | Azure/AppConfiguration-JavaScriptProvider |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | e6b91a7e6e46287d26887e3fb7a4161657bab8f7 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width