[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r29986 - trunk/subversion/libsvn_ra_serf

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Sat, 22 Mar 2008 19:08:49 -0400

lgo_at_tigris.org writes:

> --- trunk/subversion/libsvn_ra_serf/util.c Fri Mar 21 14:42:28 2008 (r29985)
> +++ trunk/subversion/libsvn_ra_serf/util.c Fri Mar 21 14:58:52 2008 (r29986)
> @@ -181,6 +181,33 @@ ssl_server_cert(void *baton, int failure
>
> return server_creds ? APR_SUCCESS : SVN_ERR_RA_SERF_SSL_CERT_UNTRUSTED;
> }
> +
> +static svn_error_t *
> +load_authorities(svn_ra_serf__connection_t *conn, const char *authorities,
> + apr_pool_t *pool)
> +{

Documentation for new internal function? (I know the ra_serf code is
documentation-deficient, but let's not make the problem worse.)

> @@ -222,6 +249,19 @@ svn_ra_serf__conn_setup(apr_socket_t *so
> {
> serf_ssl_use_default_certificates(conn->ssl_context);
> }
> + /* Are there custom CAs to load? */
> + if (conn->session->ssl_authorities)
> + {
> + svn_error_t *err;
> + err = load_authorities(conn, conn->session->ssl_authorities,
> + conn->session->pool);
> + if (err)
> + {
> + /* TODO: we need a way to pass this error back to the
> + caller */
> + svn_error_clear(err);
> + }
> + }
> #endif

Yes, we do need a way to return an error.

What about setting conn->session->pending_error? I'm no expert in the
conventions of the serf code, but it seems like that field might be
intended for exactly this purpose...

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-03-23 00:09:02 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.