You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Stein <gs...@lyra.org> on 2006/01/25 05:30:06 UTC

Re: svn commit: r18218 - in trunk: . subversion/libsvn_ra_serf

On Wed, Jan 25, 2006 at 12:03:45AM -0600, jerenkrantz@tigris.org wrote:
>...
> +++ trunk/subversion/libsvn_ra_serf/serf.c	Wed Jan 25 00:03:43 2006
>...
> +static serf_bucket_t *
> +conn_setup(apr_socket_t *sock,
> +           void *baton,
> +           apr_pool_t *pool)
> +{
> +    serf_bucket_t *b;
> +    serf_session_t *sess = baton;
> +
> +    b = serf_bucket_socket_create(sock, sess->bkt_alloc);
> +    if (sess->using_ssl) {
> +        b = serf_bucket_ssl_decrypt_create(b, sess->ssl_context,
> +                                           sess->bkt_alloc);

sess->ssl_context is NULL here.

>...

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: r18218 - in trunk: . subversion/libsvn_ra_serf

Posted by Justin Erenkrantz <ju...@erenkrantz.com>.
On 1/24/06, Greg Stein <gs...@lyra.org> wrote:
> sess->ssl_context is NULL here.

Actually, it won't be.  serf won't call conn_setup until after there
is a request to be delivered.  The ssl encryption context is only
initialized when we first create the ssl_encrypt bucket - which is
tied to the request.  I may change that in serf though.  -- justin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org