Allow specification of start_tls options in RT::Authen::ExternalAuth

RT Developers
1

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

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

2

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

3

Hi Kevin,

here you go: Human Verification

Thx

BenjaminAm 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:

Human Verification

-kevin

List info: The rt-devel Archives