<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/libgit2.git/src/streams, branch ethomson/git_obj</title>
<subtitle>github.com: libgit2/libgit2.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/'/>
<entry>
<title>stream registration: take an enum type</title>
<updated>2018-11-28T15:50:15+00:00</updated>
<author>
<name>Edward Thomson</name>
<email>ethomson@edwardthomson.com</email>
</author>
<published>2018-11-22T08:49:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=02bb39f448b9ed151a638d22fdcbccc895f4d3cf'/>
<id>02bb39f448b9ed151a638d22fdcbccc895f4d3cf</id>
<content type='text'>
Accept an enum (`git_stream_t`) during custom stream registration that
indicates whether the registration structure should be used for standard
(non-TLS) streams or TLS streams.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Accept an enum (`git_stream_t`) during custom stream registration that
indicates whether the registration structure should be used for standard
(non-TLS) streams or TLS streams.
</pre>
</div>
</content>
</entry>
<entry>
<title>stream: provide generic registration API</title>
<updated>2018-11-28T15:46:57+00:00</updated>
<author>
<name>Edward Thomson</name>
<email>ethomson@edwardthomson.com</email>
</author>
<published>2018-11-18T10:29:07+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=df2cc1087f6de8718319e5bcc65ca8e0e07b717e'/>
<id>df2cc1087f6de8718319e5bcc65ca8e0e07b717e</id>
<content type='text'>
Update the new stream registration API to be `git_stream_register`
which takes a registration structure and a TLS boolean.  This allows
callers to register non-TLS streams as well as TLS streams.

Provide `git_stream_register_tls` that takes just the init callback for
backward compatibliity.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Update the new stream registration API to be `git_stream_register`
which takes a registration structure and a TLS boolean.  This allows
callers to register non-TLS streams as well as TLS streams.

Provide `git_stream_register_tls` that takes just the init callback for
backward compatibliity.
</pre>
</div>
</content>
</entry>
<entry>
<title>http: remove cURL</title>
<updated>2018-11-28T15:46:57+00:00</updated>
<author>
<name>Edward Thomson</name>
<email>ethomson@edwardthomson.com</email>
</author>
<published>2018-10-29T10:04:48+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=21142c5a61fca0d44cbf51824dbe28f6324229e8'/>
<id>21142c5a61fca0d44cbf51824dbe28f6324229e8</id>
<content type='text'>
We previously used cURL to support HTTP proxies.  Now that we've added
this support natively, we can remove the curl dependency.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We previously used cURL to support HTTP proxies.  Now that we've added
this support natively, we can remove the curl dependency.
</pre>
</div>
</content>
</entry>
<entry>
<title>streams: remove unused tls functions</title>
<updated>2018-11-28T15:46:57+00:00</updated>
<author>
<name>Edward Thomson</name>
<email>ethomson@edwardthomson.com</email>
</author>
<published>2018-10-29T08:59:33+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=2878ad08316155bfaf7b2d7b3204aece13c9936a'/>
<id>2878ad08316155bfaf7b2d7b3204aece13c9936a</id>
<content type='text'>
The implementations of git_openssl_stream_new and
git_mbedtls_stream_new have callers protected by #ifdefs and
are never called unless compiled in.  There's no need for a
dummy implementation.  Remove them.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The implementations of git_openssl_stream_new and
git_mbedtls_stream_new have callers protected by #ifdefs and
are never called unless compiled in.  There's no need for a
dummy implementation.  Remove them.
</pre>
</div>
</content>
</entry>
<entry>
<title>tls: introduce a wrap function</title>
<updated>2018-11-28T15:46:57+00:00</updated>
<author>
<name>Edward Thomson</name>
<email>ethomson@edwardthomson.com</email>
</author>
<published>2018-10-25T07:49:01+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=43b592ac84dbd3d649022ff9503f00ecc83d5278'/>
<id>43b592ac84dbd3d649022ff9503f00ecc83d5278</id>
<content type='text'>
Introduce `git_tls_stream_wrap` which will take an existing `stream`
with an already connected socket and begin speaking TLS on top of it.
This is useful if you've built a connection to a proxy server and you
wish to begin CONNECT over it to tunnel a TLS connection.

Also update the pluggable TLS stream layer so that it can accept a
registration structure that provides an `init` and `wrap` function,
instead of a single initialization function.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Introduce `git_tls_stream_wrap` which will take an existing `stream`
with an already connected socket and begin speaking TLS on top of it.
This is useful if you've built a connection to a proxy server and you
wish to begin CONNECT over it to tunnel a TLS connection.

Also update the pluggable TLS stream layer so that it can accept a
registration structure that provides an `init` and `wrap` function,
instead of a single initialization function.
</pre>
</div>
</content>
</entry>
<entry>
<title>openssl: only say we're connected if the connection succeeded</title>
<updated>2018-11-01T04:04:29+00:00</updated>
<author>
<name>Etienne Samson</name>
<email>samson.etienne@gmail.com</email>
</author>
<published>2018-08-28T23:57:24+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=03994912795f1a6d2bd560e0bce5af64b9c0dee2'/>
<id>03994912795f1a6d2bd560e0bce5af64b9c0dee2</id>
<content type='text'>
ssl_close uses this boolean to know if SSL_shutdown should be called.
It turns out OpenSSL auto-shutdowns on failure, so if the call to
SSL_connect fails, it will complain about "shutdown while in init",
trampling the original error.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
ssl_close uses this boolean to know if SSL_shutdown should be called.
It turns out OpenSSL auto-shutdowns on failure, so if the call to
SSL_connect fails, it will complain about "shutdown while in init",
trampling the original error.
</pre>
</div>
</content>
</entry>
<entry>
<title>openssl: set the error class to GITERR_SSL</title>
<updated>2018-11-01T04:04:14+00:00</updated>
<author>
<name>Etienne Samson</name>
<email>samson.etienne@gmail.com</email>
</author>
<published>2018-08-28T23:57:22+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=caee0a6629d95669400bf6e64f64b81e5aebaa92'/>
<id>caee0a6629d95669400bf6e64f64b81e5aebaa92</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>global: replace remaining use of `git__strtol32`</title>
<updated>2018-10-18T09:58:14+00:00</updated>
<author>
<name>Patrick Steinhardt</name>
<email>ps@pks.im</email>
</author>
<published>2018-10-18T09:58:14+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=2613fbb26a3e1a34dda8a5d198c108626cfd6cc3'/>
<id>2613fbb26a3e1a34dda8a5d198c108626cfd6cc3</id>
<content type='text'>
Replace remaining uses of the `git__strtol32` function. While these uses
are all safe as the strings were either sanitized or from a trusted
source, we want to remove `git__strtol32` altogether to avoid future
misuse.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Replace remaining uses of the `git__strtol32` function. While these uses
are all safe as the strings were either sanitized or from a trusted
source, we want to remove `git__strtol32` altogether to avoid future
misuse.
</pre>
</div>
</content>
</entry>
<entry>
<title>stransport: fix a warning on iOS</title>
<updated>2018-09-25T12:44:40+00:00</updated>
<author>
<name>Etienne Samson</name>
<email>samson.etienne@gmail.com</email>
</author>
<published>2018-03-29T22:20:23+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=21496c3024b3ab4168a8e4b4d1ef03da736aeac5'/>
<id>21496c3024b3ab4168a8e4b4d1ef03da736aeac5</id>
<content type='text'>
"warning: values of type 'OSStatus' should not be used as format arguments; add an explicit cast to 'int' instead [-Wformat]"</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
"warning: values of type 'OSStatus' should not be used as format arguments; add an explicit cast to 'int' instead [-Wformat]"</pre>
</div>
</content>
</entry>
<entry>
<title>mbedtls: remove unused variable "cacert"</title>
<updated>2018-07-26T10:11:34+00:00</updated>
<author>
<name>Patrick Steinhardt</name>
<email>ps@pks.im</email>
</author>
<published>2018-07-26T10:11:34+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libgit2.git/commit/?id=d4198d4d7e9bf1729b1613c0d05e0abb6a5a3c6d'/>
<id>d4198d4d7e9bf1729b1613c0d05e0abb6a5a3c6d</id>
<content type='text'>
In commit 382ed1e87 (mbedtls: load default CA certificates, 2018-03-29),
the function `git_mbedtls_stream_global_init` was refactored to call out
to `git_mbedtls__set_cert_location` instead of setting up the
certificates itself. The conversion forgot to remove the now-unused
"cacert" variable, which is now only getting declared to be free'd at
the end of the function. Remove it.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
In commit 382ed1e87 (mbedtls: load default CA certificates, 2018-03-29),
the function `git_mbedtls_stream_global_init` was refactored to call out
to `git_mbedtls__set_cert_location` instead of setting up the
certificates itself. The conversion forgot to remove the now-unused
"cacert" variable, which is now only getting declared to be free'd at
the end of the function. Remove it.
</pre>
</div>
</content>
</entry>
</feed>
