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.

Sign up for GitHub

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

Jump to bottom

bug: nil pointer if backedtls.minVersion is set but backedtls.maxVersion is not #3834

Closed
guydc opened this issue Jul 12, 2024 · 0 comments · Fixed by #3835
Closed

bug: nil pointer if backedtls.minVersion is set but backedtls.maxVersion is not #3834

guydc opened this issue Jul 12, 2024 · 0 comments · Fixed by #3835
Assignees
@guydc
Labels
kind/bug Something isn't working

Comments

@guydc
Copy link
Contributor

guydc commented Jul 12, 2024

Description:
nil pointer if backedtls.minVersion is set but backedtls.maxVersion is not:

gateway/internal/gatewayapi/backendtlspolicy.go

Lines 72 to 73 in b9d1493

if envoyProxy.Spec.BackendTLS.MinVersion != nil {
tlsBundle.MaxVersion = ptr.To(ir.TLSVersion(*envoyProxy.Spec.BackendTLS.MaxVersion))

@guydc guydc added the kind/bug Something isn't working label Jul 12, 2024
@guydc guydc self-assigned this Jul 12, 2024
@guydc guydc mentioned this issue Jul 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@guydc guydc

Labels
kind/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: backendtls minversion guydc/gateway
1 participant
@guydc