You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shiro.apache.org by Paul Merlin <es...@n0pe.org> on 2010/06/11 23:36:19 UTC

Re: X509Certificate support

Hey,

Now that both the release and the logo contest (the winner is awesome BTW) are 
behind us, could someone from the core team take a serious look at my patch and 
tell my what we'll do about it ?

Best regards

/Paul


Le lundi 24 mai 2010 19:23:14, Brian Demers a écrit :
> Also,  After taking a very quick look at the patch.
> 
> This requires a dependency on bouncycastle ( sort of the obvious choice ).
>  But this means the core will depend on bouncycastle.
> 
> Les, what is the _policy_ for this type of dependency,   I don't think it
> should be a core dep.
> 
> A couple of options are:
> 1.) Push this realm to its own sub-module.
> 2.) Mark the dep as optional/provided/etc ( not a big fan of this one )
> 3.) Nothing, leave it as a dep


Re: X509Certificate support

Posted by Les Hazlewood <lh...@apache.org>.
Hi Paul,

Yep, I will have some time for this next week.  Thanks for being
patient through the last hectic month! ;)

Les

On Fri, Jun 11, 2010 at 2:36 PM, Paul Merlin <es...@n0pe.org> wrote:
> Hey,
>
> Now that both the release and the logo contest (the winner is awesome BTW) are
> behind us, could someone from the core team take a serious look at my patch and
> tell my what we'll do about it ?
>
> Best regards
>
> /Paul
>
>
> Le lundi 24 mai 2010 19:23:14, Brian Demers a écrit :
>> Also,  After taking a very quick look at the patch.
>>
>> This requires a dependency on bouncycastle ( sort of the obvious choice ).
>>  But this means the core will depend on bouncycastle.
>>
>> Les, what is the _policy_ for this type of dependency,   I don't think it
>> should be a core dep.
>>
>> A couple of options are:
>> 1.) Push this realm to its own sub-module.
>> 2.) Mark the dep as optional/provided/etc ( not a big fan of this one )
>> 3.) Nothing, leave it as a dep
>
>