From dd20b8adb65c2d974ee8c04fe34fd6b3aac36b97 Mon Sep 17 00:00:00 2001
From: Aleh Zasypkin <aleh.zasypkin@gmail.com>
Date: Thu, 24 Jun 2021 18:22:14 +0200
Subject: [PATCH] Avoid using deprecated camelCase parameters for SAML APIs.
 (#103091)

---
 .../security/server/authentication/providers/saml.test.ts | 8 ++++----
 .../security/server/authentication/providers/saml.ts      | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/x-pack/plugins/security/server/authentication/providers/saml.test.ts b/x-pack/plugins/security/server/authentication/providers/saml.test.ts
index dfcdb66e61c351..4a32383d18deca 100644
--- a/x-pack/plugins/security/server/authentication/providers/saml.test.ts
+++ b/x-pack/plugins/security/server/authentication/providers/saml.test.ts
@@ -1186,7 +1186,7 @@ describe('SAMLAuthenticationProvider', () => {
       expect(mockOptions.client.asInternalUser.transport.request).toHaveBeenCalledWith({
         method: 'POST',
         path: '/_security/saml/invalidate',
-        body: { queryString: 'SAMLRequest=xxx%20yyy', realm: 'test-realm' },
+        body: { query_string: 'SAMLRequest=xxx%20yyy', realm: 'test-realm' },
       });
     });
 
@@ -1286,7 +1286,7 @@ describe('SAMLAuthenticationProvider', () => {
       expect(mockOptions.client.asInternalUser.transport.request).toHaveBeenCalledWith({
         method: 'POST',
         path: '/_security/saml/invalidate',
-        body: { queryString: 'SAMLRequest=xxx%20yyy', realm: 'test-realm' },
+        body: { query_string: 'SAMLRequest=xxx%20yyy', realm: 'test-realm' },
       });
     });
 
@@ -1305,7 +1305,7 @@ describe('SAMLAuthenticationProvider', () => {
       expect(mockOptions.client.asInternalUser.transport.request).toHaveBeenCalledWith({
         method: 'POST',
         path: '/_security/saml/invalidate',
-        body: { queryString: 'SAMLRequest=xxx%20yyy', realm: 'test-realm' },
+        body: { query_string: 'SAMLRequest=xxx%20yyy', realm: 'test-realm' },
       });
     });
 
@@ -1324,7 +1324,7 @@ describe('SAMLAuthenticationProvider', () => {
       expect(mockOptions.client.asInternalUser.transport.request).toHaveBeenCalledWith({
         method: 'POST',
         path: '/_security/saml/invalidate',
-        body: { queryString: 'SAMLRequest=xxx%20yyy', realm: 'test-realm' },
+        body: { query_string: 'SAMLRequest=xxx%20yyy', realm: 'test-realm' },
       });
     });
 
diff --git a/x-pack/plugins/security/server/authentication/providers/saml.ts b/x-pack/plugins/security/server/authentication/providers/saml.ts
index ea818e5df6e123..37e7e868e4d3da 100644
--- a/x-pack/plugins/security/server/authentication/providers/saml.ts
+++ b/x-pack/plugins/security/server/authentication/providers/saml.ts
@@ -624,9 +624,9 @@ export class SAMLAuthenticationProvider extends BaseAuthenticationProvider {
       await this.options.client.asInternalUser.transport.request({
         method: 'POST',
         path: '/_security/saml/invalidate',
-        // Elasticsearch expects `queryString` without leading `?`, so we should strip it with `slice`.
+        // Elasticsearch expects `query_string` without leading `?`, so we should strip it with `slice`.
         body: {
-          queryString: request.url.search ? request.url.search.slice(1) : '',
+          query_string: request.url.search ? request.url.search.slice(1) : '',
           realm: this.realm,
         },
       })