Re: [PR] Add verfied chain

2020-05-20 Thread William Lallemand
On Mon, May 18, 2020 at 08:09:36PM +0200, William Dauchy wrote: > Hi Arjen, > > On Mon, May 18, 2020 at 6:02 PM Arjen Nienhuis wrote: > > I used PKCS7 because I did not know how to parse concatenated blobs. > > Mathilde, how did we planned to use it? :) > > > I think you should use

Re: [PR] Add verfied chain

2020-05-18 Thread William Dauchy
Hi Arjen, On Mon, May 18, 2020 at 6:02 PM Arjen Nienhuis wrote: > I used PKCS7 because I did not know how to parse concatenated blobs. Mathilde, how did we planned to use it? :) > I think you should use SSL_get_peer_cert_chain because: > - BoringSSL has no SSL_get0_verified_chain. > - For

Re: [PR] Add verfied chain

2020-05-18 Thread Arjen Nienhuis
On 18-05-2020 15:57, William Lallemand wrote: On Mon, May 18, 2020 at 02:49:28PM +0200, William Dauchy wrote: Hello William L., On Wed, Dec 4, 2019 at 4:24 PM PR Bot wrote: Patch title(s): MINOR: add fetch 'ssl_c_verified_chain' Merge branch 'master' of

Re: [PR] Add verfied chain

2020-05-18 Thread Emeric Brun
Hi All, On 5/18/20 4:32 PM, William Dauchy wrote: > On Mon, May 18, 2020 at 3:58 PM William Lallemand > wrote: >> I suppose it was put in a PKCS7 container to be able to distinguish each >> DER part of the chain easily? So It can be used by an external tool. I'm not >> sure of what is done with

Re: [PR] Add verfied chain

2020-05-18 Thread William Dauchy
On Mon, May 18, 2020 at 3:58 PM William Lallemand wrote: > I suppose it was put in a PKCS7 container to be able to distinguish each > DER part of the chain easily? So It can be used by an external tool. I'm not > sure of what is done with the result of this. > > The two patches seem to have

Re: [PR] Add verfied chain

2020-05-18 Thread William Lallemand
On Mon, May 18, 2020 at 02:49:28PM +0200, William Dauchy wrote: > Hello William L., > > On Wed, Dec 4, 2019 at 4:24 PM PR Bot wrote: > > Patch title(s): > >MINOR: add fetch 'ssl_c_verified_chain' > >Merge branch 'master' of https://github.com/haproxy/haproxy > > Link: > >

Re: [PR] Add verfied chain

2020-05-18 Thread William Dauchy
Hello William L., On Wed, Dec 4, 2019 at 4:24 PM PR Bot wrote: > Patch title(s): >MINOR: add fetch 'ssl_c_verified_chain' >Merge branch 'master' of https://github.com/haproxy/haproxy > Link: >https://github.com/haproxy/haproxy/pull/396 > Edit locally: >wget

[PR] Add verfied chain

2019-12-04 Thread PR Bot
Dear list! Author: Arjen Nienhuis Number of patches: 2 This is an automated relay of the Github pull request: Add verfied chain Patch title(s): MINOR: add fetch 'ssl_c_verified_chain' Merge branch 'master' of https://github.com/haproxy/haproxy Link: