Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[kibana] optionally disable SA token automount (#1301) #1481

Merged
merged 1 commit into from
Dec 14, 2021

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Dec 14, 2021

Kibana has no direct interaction with the Kubernetes API, and as such,
it does not need a mounted service account token in its pods. By
disabling this automount, potential attackers cannot access the API on
behalf/through the Pod.

This commit allows users to opt out on SA token automount, but leaves
leaves current behaviour untouched to avoid breaking things.

Signed-off-by: Jorik Jonker jorik.jonker@eu.equinix.com

Kibana has no direct interaction with the Kubernetes API, and as such,
it does not need a mounted service account token in its pods. By
disabling this automount, potential attackers cannot access the API on
behalf/through the Pod.

This commit allows users to opt out on SA token automount, but leaves
leaves current behaviour untouched to avoid breaking things.

Signed-off-by: Jorik Jonker <jorik.jonker@eu.equinix.com>
@jmlrt jmlrt added the backport label Dec 14, 2021
@jmlrt jmlrt requested review from framsouza and a team December 14, 2021 12:03
@cla-checker-service
Copy link

cla-checker-service bot commented Dec 14, 2021

💚 CLA has been signed

@jmlrt jmlrt changed the base branch from main to 7.x December 14, 2021 12:04
@jmlrt jmlrt changed the title backport 1301 7x [kibana] optionally disable SA token automount (#1301) Dec 14, 2021
@jmlrt
Copy link
Member Author

jmlrt commented Dec 14, 2021

cla/check

@jmlrt jmlrt merged commit bf016ca into elastic:7.x Dec 14, 2021
@jmlrt jmlrt deleted the backport-1301-7x branch December 14, 2021 12:44
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
@jmlrt jmlrt mentioned this pull request Apr 21, 2022
This was referenced Sep 14, 2022
@cospeedster
Copy link
Contributor

@jmlrt Can we backport this to 7.13.x?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants