Skip to content

fix!: remove Namespace field from AuthProxyWorkloadSelector#168

Merged
hessjcg merged 1 commit intomainfrom
remove-fields
Jan 3, 2023
Merged

fix!: remove Namespace field from AuthProxyWorkloadSelector#168
hessjcg merged 1 commit intomainfrom
remove-fields

Conversation

@hessjcg
Copy link
Copy Markdown
Collaborator

@hessjcg hessjcg commented Dec 15, 2022

For security, AuthProxyWorkload resources are only allowed to affect workloads running
in the same namespace. This prevents one AuthProxyWorkload in one namespace from affecting
a workload in a different namespace.

This change removes the WorkloadSelectorSpec.Namespace field and associated code that allowed
cross-namespace workload selection.

@hessjcg hessjcg requested a review from a team December 15, 2022 17:44
@hessjcg hessjcg changed the title fix: remove Namespace field from AuthProxyWorkloadSelector fix!: remove Namespace field from AuthProxyWorkloadSelector Dec 15, 2022
@hessjcg hessjcg merged commit 7bcc27d into main Jan 3, 2023
@hessjcg hessjcg deleted the remove-fields branch January 3, 2023 18:50
hessjcg pushed a commit that referenced this pull request Jan 18, 2023
0.2.0 (2023-01-18)

BREAKING CHANGES
- remove Namespace field from AuthProxyWorkloadSelector (#168)

Bug Fixes
- remove Namespace field from AuthProxyWorkloadSelector (#168) (7bcc27d)
- Update installer.sh to use helm for cert-manager (#163) (62fc5dc), closes #157

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Release PR Generate Bot action release-please[bot] <release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants