users@jersey.java.net

Re: [Jersey] HTTPSProperties issue

From: Pavel Bucek <Pavel.Bucek_at_Sun.COM>
Date: Thu, 15 Apr 2010 11:21:24 +0200

Hello John,

I agree, you should not be able to set null SSLContext.

I'll update javadoc and code, constructor will now throw
IllegalArgumentException when you try to do that.

Thanks,
Pavel


John Calcote wrote:
> Hi Paul,
>
> Just ran into a usability bug. Thought I'd drop you a note about it.
>
> When I created my HTTPProperties object, I wanted to supply a host
> name verifier, and possibly an ssl context. I assumed I could pass a
> null ssl context, meaning that the constructor would then use a
> default ssl context. However the HTTPSProperties constructor simply
> stores the null, and then crashes later. Might be nice to either
> document the fact that null is not acceptable in this ctor, or change
> the ctor to accept null and then build a default ssl context.
>
> Thanks,
> John
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe_at_jersey.dev.java.net
> For additional commands, e-mail: users-help_at_jersey.dev.java.net
>