Skip to content

[7.17] Fix double sending of response in TransportOpenIdConnectPrepareAuthenticationAction (#89930)#89953

Merged
elasticsearchmachine merged 2 commits intoelastic:7.17from
original-brownbear:backport/7.17/pr-89930
Sep 9, 2022
Merged

[7.17] Fix double sending of response in TransportOpenIdConnectPrepareAuthenticationAction (#89930)#89953
elasticsearchmachine merged 2 commits intoelastic:7.17from
original-brownbear:backport/7.17/pr-89930

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.17:

…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 cf9e982 into elastic:7.17 Sep 9, 2022
@original-brownbear original-brownbear deleted the backport/7.17/pr-89930 branch September 9, 2022 11:43
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 v7.17.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants