Skip to content

Commit

Permalink
Add SNI support to ESP for https clients
Browse files Browse the repository at this point in the history
This PR adds the SNI (Server Name Indication) support to ESP for https clients.
Problem solved by this PR: before this PR, ESP https clients do not support SNI for https connections. Details in the following two github issues.
- cloudendpoints#271
- cloudendpoints#262
  • Loading branch information
lei-tang committed Jun 10, 2018
1 parent 8534d96 commit a8787fd
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/nginx/http.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1002,6 +1002,8 @@ Status initialize_upstream_request(ngx_log_t *log, HTTPRequest *request,
http_cctx->main_conf[ngx_esp_module.ctx_index]);
http_connection->upstream_conf.ssl = mc->ssl;
http_connection->upstream_conf.ssl_session_reuse = 1;
// For SNI support
http_connection->upstream_conf.ssl_server_name = 1;
if (mc->cert_path.len > 0) {
http_connection->upstream_conf.ssl_verify = 1;
}
Expand Down

0 comments on commit a8787fd

Please sign in to comment.