hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Julius Davies" <juliusdav...@gmail.com>
Subject Re: [https/ssl] - CN verification: setAllowedNames() idea
Date Tue, 20 Feb 2007 15:09:53 GMT
Hi, Roland,

Thanks for looking at this, and thanks for your suggestion.  I'm going
to do that.  Thanks!  I got distracted by:

SSLSocket.setEnabledCipherSuites(String[] suites)
SSLSocket.setEnabledProtocols(String[] protocols)

:-)


yours,

Julius



On 2/18/07, Roland Weber <http-async@dubioso.net> wrote:
> Hi Julius,
>
> > public void setAllowedNames( String[] allowedNames )
> > {
> >     this.allowedNames = allowedNames;
> > }
>
> I like the idea. But you might want to keep the names in
> an internal collection (ArrayList/LinkedList) and offer
>   addAllowedName(String)
>   addAllowedNames(String[])
>   clearAllowedNames()
> instead of passing in an array. That makes sure that the
> internal data structure is not modified uncontrolled.
>
> cheers,
>   Roland
>

-- 
yours,

Julius Davies
416-652-0183
http://juliusdavies.ca/

---------------------------------------------------------------------
To unsubscribe, e-mail: httpcomponents-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: httpcomponents-dev-help@jakarta.apache.org


Mime
View raw message