Bugzilla – Attachment 21282 Details for
Bug 10925
LDAP Auth fails if DEBUG is enable
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10925: LDAP auth fails if DEBUG is enable
Bug-10925-LDAP-auth-fails-if-DEBUG-is-enable.patch (text/plain), 1.61 KB, created by
Jonathan Druart
on 2013-09-20 15:38:21 UTC
(
hide
)
Description:
Bug 10925: LDAP auth fails if DEBUG is enable
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2013-09-20 15:38:21 UTC
Size:
1.61 KB
patch
obsolete
>From 431c457a4c7e51e0c61d1e825c140f87115002b0 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Fri, 20 Sep 2013 17:21:40 +0200 >Subject: [PATCH] Bug 10925: LDAP auth fails if DEBUG is enable > >To reproduce: >1/ Edit your apache virtual host and set the DEBUG environment variable >(SetEnv DEBUG 1). >2/ Try to login with an ldap user >3/ You will be redirected to the 500 error page. >The Koha logs contains: >malformed header from script. Bad header=------------------------------: mainpage.pl > >The hashdump routine directly punks to the STDOUT (!) and breaks the >headers. >It appears Net::LDAP::?->dump does the same thing. >--- > C4/Auth_with_ldap.pm | 3 --- > 1 file changed, 3 deletions(-) > >diff --git a/C4/Auth_with_ldap.pm b/C4/Auth_with_ldap.pm >index d7a5e9a..0efeb95 100644 >--- a/C4/Auth_with_ldap.pm >+++ b/C4/Auth_with_ldap.pm >@@ -26,7 +26,6 @@ use C4::Context; > use C4::Members qw(AddMember changepassword); > use C4::Members::Attributes; > use C4::Members::AttributeTypes; >-use C4::Utils qw( :all ); > use List::MoreUtils qw( any ); > use Net::LDAP; > use Net::LDAP::Filter; >@@ -203,10 +202,8 @@ sub ldap_entry_2_hash { > my %memberhash; > $userldapentry->exists('uid'); # This is bad, but required! By side-effect, this initializes the attrs hash. > if ($debug) { >- print STDERR "\nkeys(\%\$userldapentry) = " . join(', ', keys %$userldapentry), "\n", $userldapentry->dump(); > foreach (keys %$userldapentry) { > print STDERR "\n\nLDAP key: $_\t", sprintf('(%s)', ref $userldapentry->{$_}), "\n"; >- hashdump("LDAP key: ",$userldapentry->{$_}); > } > } > my $x = $userldapentry->{attrs} or return; >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 10925
:
21282
|
21545
|
21685