Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

envoyconfig: prevent nil reproxy handler #3345

Merged
merged 1 commit into from
May 11, 2022
Merged

Conversation

wasaga
Copy link
Contributor

@wasaga wasaga commented May 11, 2022

Summary

Reproxy handler is set to nil during config validation in the console and ingress controller,
that would result in SIGSEGV if kubernetes_service_account_token config option is set.

as reproxy package is internal, a fix is to prevent a nil reproxy.

Related issues

pomerium/ingress-controller#205

Checklist

  • reference any related issues
  • updated docs
  • updated unit tests
  • updated UPGRADING.md
  • add appropriate tag (improvement / bug / etc)
  • ready for review

@wasaga wasaga requested a review from a team as a code owner May 11, 2022 03:17
@wasaga wasaga requested a review from calebdoxsey May 11, 2022 03:17
@wasaga wasaga added bug Something isn't working backport 0-17-0 labels May 11, 2022
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.006%) to 64.325% when pulling 90f0bd8 on wasaga/init-reproxy-handler into 2b819b3 on main.

@wasaga wasaga merged commit a15106e into main May 11, 2022
@wasaga wasaga deleted the wasaga/init-reproxy-handler branch May 11, 2022 16:17
backport-actions-token bot pushed a commit that referenced this pull request May 11, 2022
(cherry picked from commit a15106e)
calebdoxsey pushed a commit that referenced this pull request May 11, 2022
(cherry picked from commit a15106e)

Co-authored-by: Denis Mishin <dmishin@pomerium.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 0-17-0 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants