> On Nov 17, 2014, at 4:09 PM, Sudheer Vinukonda 
> <sudhe...@yahoo-inc.com.INVALID> wrote:
> 
> Hi All,
> 
> I would like add a new API TSSslAdvertiseProtocolSet(), to address the
> requirement of modifying the NPN advertising list for each SSL connection
> based on SNI (please refer TS-3153).
> 
> https://issues.apache.org/jira/browse/TS-3153
> 
> 
> Below is a brief description of the API. Please review and provide
> comments/concerns.
> 
> 
> 
> +TSSslAdvertiseProtocolSet
> +============
> +
> +Synopsis
> +--------
> +
> +`#include <ts/ts.h>`
> +
> +.. c:function:: TSReturnCode TSSslAdvertiseProtocolSet(TSVConn sslp,
> const unsigned char ** list, unsigned int count);

In the rest of the APIs, we use the term "named protocol". This API does not 
let you set arbitrary protocol names, so I'd suggest

TSReturnCode TSSslFilterNamedProtocols()

> +
> +Description
> +-----------
> +
> +   Modifies the NPN advertisement list for a given SSL connection with
> :arg:`list`.

What is list? In what way is the advertisement list modified?

> If :arg:`count` is 0, sets the NPN advertisement list to the
> default registered protocol list for the end point.

Isn't the default already the default? If list is NULL or count is 0, just 
return an error.

> Note that, the plugin
> that uses this API owns the :arg:`list` and is responsible for making sure
> it points to a valid memory.

Not sure what you are saying here? I guess we should never pass invalid memory 
anywhere, right?

> +
> 
> 
> 
> Thanks,
> 
> Sudheer
> 

Reply via email to