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

internal/contour: generate an Envoy secret for the fallback certificate #2723

Merged
merged 1 commit into from
Jul 23, 2020

Conversation

jpeach
Copy link
Contributor

@jpeach jpeach commented Jul 23, 2020

Update the DAG visitor to emit an Envoy secret if the Contour fallback
certificate is used by a secure virtual host.

This fixes #2720.

Signed-off-by: James Peach jpeach@vmware.com

Update the DAG visitor to emit an Envoy secret if the Contour fallback
certificate is used by a secure virtual host.

This fixes projectcontour#2720.

Signed-off-by: James Peach <jpeach@vmware.com>
Copy link
Member

@youngnick youngnick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link

codecov bot commented Jul 23, 2020

Codecov Report

Merging #2723 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2723      +/-   ##
==========================================
+ Coverage   76.88%   76.90%   +0.01%     
==========================================
  Files          72       72              
  Lines        5711     5715       +4     
==========================================
+ Hits         4391     4395       +4     
  Misses       1228     1228              
  Partials       92       92              
Impacted Files Coverage Δ
internal/contour/secret.go 100.00% <100.00%> (ø)

@jpeach jpeach merged commit 5c79158 into projectcontour:master Jul 23, 2020
@jpeach jpeach deleted the emit-fallback-secret branch July 23, 2020 04:18
@ravilr
Copy link

ravilr commented Jul 27, 2020

this was pointed during the original PR review here: #2477 (comment)

tthebst pushed a commit to tthebst/contour that referenced this pull request Aug 6, 2020
…te (projectcontour#2723)

Update the DAG visitor to emit an Envoy secret if the Contour fallback
certificate is used by a secure virtual host.

This fixes projectcontour#2720.

Signed-off-by: James Peach <jpeach@vmware.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.

Httpproxy configured with fallback certificate fails when SNI not included
3 participants