Skip to content

Commit

Permalink
mod_proxy: we don't need ap_proxy_ssl_connection_cleanup() anymore with
Browse files Browse the repository at this point in the history
ap_proxy_check_backend() used at connection reuse time, so remove its last call and deprecate it.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1750416 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information
ylavic committed Jun 27, 2016
1 parent c0f3538 commit b803da8
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 5 deletions.
1 change: 1 addition & 0 deletions modules/proxy/mod_proxy.h
Original file line number Diff line number Diff line change
Expand Up @@ -668,6 +668,7 @@ PROXY_DECLARE(int) ap_proxy_checkproxyblock(request_rec *r, proxy_server_conf *c
PROXY_DECLARE(int) ap_proxy_pre_http_request(conn_rec *c, request_rec *r);
/* DEPRECATED (will be replaced with ap_proxy_connect_backend */
PROXY_DECLARE(int) ap_proxy_connect_to_backend(apr_socket_t **, const char *, apr_sockaddr_t *, const char *, proxy_server_conf *, request_rec *);
/* DEPRECATED (will be replaced with ap_proxy_check_backend */
PROXY_DECLARE(apr_status_t) ap_proxy_ssl_connection_cleanup(proxy_conn_rec *conn,
request_rec *r);
PROXY_DECLARE(int) ap_proxy_ssl_enable(conn_rec *c);
Expand Down
5 changes: 0 additions & 5 deletions modules/proxy/mod_proxy_http.c
Original file line number Diff line number Diff line change
Expand Up @@ -1994,13 +1994,8 @@ static int proxy_http_handler(request_rec *r, proxy_worker *worker,
worker, r->server)) != OK)
goto cleanup;


backend->is_ssl = is_ssl;

if (is_ssl) {
ap_proxy_ssl_connection_cleanup(backend, r);
}

/*
* In the case that we are handling a reverse proxy connection and this
* is not a request that is coming over an already kept alive connection
Expand Down
1 change: 1 addition & 0 deletions modules/proxy/proxy_util.c
Original file line number Diff line number Diff line change
Expand Up @@ -1399,6 +1399,7 @@ static apr_status_t connection_cleanup(void *theconn)
return APR_SUCCESS;
}

/* DEPRECATED */
PROXY_DECLARE(apr_status_t) ap_proxy_ssl_connection_cleanup(proxy_conn_rec *conn,
request_rec *r)
{
Expand Down

0 comments on commit b803da8

Please sign in to comment.