Re: [API] Proposed removal from TSAPI

2018-05-12 Thread Masakazu Kitajo
+1 On Fri, May 11, 2018 at 6:29 PM, Alan Carroll < solidwallofc...@oath.com.invalid> wrote: > +1 > > On Thu, May 10, 2018 at 9:25 AM, Bryan Call wrote: > > > +1 > > > > -Bryan > > > > > On May 10, 2018, at 2:05 PM, Susan Hinrichs > > > wrote: > > > > > > I just discovered the TS API > > > > > >

Re: [API] Proposed removal from TSAPI

2018-05-11 Thread Alan Carroll
+1 On Thu, May 10, 2018 at 9:25 AM, Bryan Call wrote: > +1 > > -Bryan > > > On May 10, 2018, at 2:05 PM, Susan Hinrichs > wrote: > > > > I just discovered the TS API > > > > void *TSHttpSsnConnectionGet(TSHttpSsn ssnp) > > > > This seems redundant with the TSVConnSSLConnectionGet and > > TSHttp

Re: [API] Proposed removal from TSAPI

2018-05-10 Thread Bryan Call
+1 -Bryan > On May 10, 2018, at 2:05 PM, Susan Hinrichs wrote: > > I just discovered the TS API > > void *TSHttpSsnConnectionGet(TSHttpSsn ssnp) > > This seems redundant with the TSVConnSSLConnectionGet and > TSHttpSsnClientVConnGet functions. > > I propose removing it. > > If we need to ke

Re: [API] Proposed removal from TSAPI

2018-05-10 Thread Leif Hedstrom
> On May 10, 2018, at 2:46 PM, Susan Hinrichs wrote: > > Typo'd the name. Should be > > void *TSHttpSsnSSLConnectionGet(TSHttpSsn ssnp) > Yeh, we should nuke this in favor of the two finer granular APIs. +1 — Leif

Re: [API] Proposed removal from TSAPI

2018-05-10 Thread Susan Hinrichs
Typo'd the name. Should be void *TSHttpSsnSSLConnectionGet(TSHttpSsn ssnp) On Thu, May 10, 2018 at 2:05 PM, Susan Hinrichs wrote: > I just discovered the TS API > > void *TSHttpSsnConnectionGet(TSHttpSsn ssnp) > > This seems redundant with the TSVConnSSLConnectionGet and > TSHttpSsnClientVConn

[API] Proposed removal from TSAPI

2018-05-10 Thread Susan Hinrichs
I just discovered the TS API void *TSHttpSsnConnectionGet(TSHttpSsn ssnp) This seems redundant with the TSVConnSSLConnectionGet and TSHttpSsnClientVConnGet functions. I propose removing it. If we need to keep this, we should document the API and change the signature to return TSSslConnection in