Skip to content

[8.4] Fix double sending of response in TransportOpenIdConnectPrepareAuthenticationAction (#89930)#89954

Merged
elasticsearchmachine merged 1 commit intoelastic:8.4from
original-brownbear:backport/8.4/pr-89930
Sep 9, 2022
Merged

[8.4] Fix double sending of response in TransportOpenIdConnectPrepareAuthenticationAction (#89930)#89954
elasticsearchmachine merged 1 commit intoelastic:8.4from
original-brownbear:backport/8.4/pr-89930

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

Backports the following commits to 8.4:

…ticationAction (#89930)

This fixes an obvious bug where the listener was resolved twice if any of the first
two failure conditions in the changed method were met.
Prior to #89873 this would lead to a memory leak.
@original-brownbear original-brownbear added :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Security Meta label for security team labels Sep 9, 2022
@elasticsearchmachine elasticsearchmachine merged commit 02e0c8f into elastic:8.4 Sep 9, 2022
@original-brownbear original-brownbear deleted the backport/8.4/pr-89930 branch September 9, 2022 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team v8.4.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants