Mailing List Archive: Request Tracker: Devel

[Patch] Allow specification of start_tls options in RT::Authen::ExternalAuth

Index | Next | Previous | View Threaded

mailinglists at boksa

Jun 5, 2009, 7:27 AM

Post #1 of 3 (696 views)
Permalink

[Patch] Allow specification of start_tls options in RT::Authen::ExternalAuth

Dear developers,

I am in the process of setting up RT 3.8.3. The LDAP server I am
working with requires a client certificate when using TLS, so I had to
modify RT::Authen::ExternalAuth (against trunk in Rev. 19912). Hope I
did everything right (espacially regarding the format of the patch) -
would be glad to get some feedback, please feel free to include the
patch in the next release.

Thanks for developing such a powerful tool.

Regards

Benjamin

Attachments:

RT-Authen-ExternalAuth-19912-start_tls-options.patch (2.40 KB)

falcone at bestpractical

Jun 8, 2009, 9:44 AM

Post #2 of 3 (607 views)
Permalink

Re: [Patch] Allow specification of start_tls options in RT::Authen::ExternalAuth [In reply to]

On Jun 5, 2009, at 10:27 AM, Benjamin Boksa wrote:

> I am in the process of setting up RT 3.8.3. The LDAP server I am
> working with requires a client certificate when using TLS, so I had
> to modify RT::Authen::ExternalAuth (against trunk in Rev. 19912).
> Hope I did everything right (espacially regarding the format of the
> patch) - would be glad to get some feedback, please feel free to
> include the patch in the next release.

This looks ok to me, but I don't have an ldaps to test it against

RT-Authen-ExternalAuth is maintained separately from RT, so you should
probably send a copy of this patch to the rt.cpan.org queue,
directions are here:

http://rt.cpan.org/Public/Bug/Report.html?Queue=RT-Authen-ExternalAuth

-kevin
_______________________________________________
List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel

mailinglists at boksa

Jun 12, 2009, 2:55 PM

Post #3 of 3 (562 views)
Permalink

Re: [Patch] Allow specification of start_tls options in RT::Authen::ExternalAuth [In reply to]

Hi Kevin,

here you go: https://rt.cpan.org/Ticket/Display.html?id=46900

Thx

Benjamin

Am 08.06.2009 um 18:44 schrieb Kevin Falcone:

> On Jun 5, 2009, at 10:27 AM, Benjamin Boksa wrote:
>
>> I am in the process of setting up RT 3.8.3. The LDAP server I am
>> working with requires a client certificate when using TLS, so I had
>> to modify RT::Authen::ExternalAuth (against trunk in Rev. 19912).
>> Hope I did everything right (espacially regarding the format of the
>> patch) - would be glad to get some feedback, please feel free to
>> include the patch in the next release.
>
> This looks ok to me, but I don't have an ldaps to test it against
>
> RT-Authen-ExternalAuth is maintained separately from RT, so you should
> probably send a copy of this patch to the rt.cpan.org queue,
> directions are here:
>
> http://rt.cpan.org/Public/Bug/Report.html?Queue=RT-Authen-ExternalAuth
>
> -kevin
> _______________________________________________
> List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel

_______________________________________________
List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact lists@gossamer-threads.com