Explicitly ignoring a 429 from triggering a license refresh

kobelb · kobelb · commit 51fbd969c8ec · 2020-09-15T10:56:00.000-07:00
diff --git a/x-pack/plugins/licensing/server/on_pre_response_handler.ts b/x-pack/plugins/licensing/server/on_pre_response_handler.ts
@@ -17,7 +17,7 @@ export function createOnPreResponseHandler(
     // Elasticsearch in case the error is due to a change in license information
     // in Elasticsearch.
     // https://github.com/elastic/x-pack-kibana/pull/2876
-    if (res.statusCode >= 400) {
+    if (res.statusCode >= 400 && res.statusCode !== 429) {
       await refresh();
     }
     const license = await license$.pipe(take(1)).toPromise();

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ export function createOnPreResponseHandler(`
`17`	`17`	`// Elasticsearch in case the error is due to a change in license information`
`18`	`18`	`// in Elasticsearch.`
`19`	`19`	`// https://github.com/elastic/x-pack-kibana/pull/2876`
`20`		`- if (res.statusCode >= 400) {`
	`20`	`+ if (res.statusCode >= 400 && res.statusCode !== 429) {`
`21`	`21`	`await refresh();`
`22`	`22`	`}`
`23`	`23`	`const license = await license$.pipe(take(1)).toPromise();`