Bugzilla – Attachment 10449 Details for
Bug 8279
CAS Debugging improvements
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 8279: CAS Debugging improvements
Bug-8279-CAS-Debugging-improvements.patch (text/plain), 1.46 KB, created by
Chris Cormack
on 2012-06-24 08:02:34 UTC
(
hide
)
Description:
Bug 8279: CAS Debugging improvements
Filename:
MIME Type:
Creator:
Chris Cormack
Created:
2012-06-24 08:02:34 UTC
Size:
1.46 KB
patch
obsolete
>From d7fbb8c7c1ecabb51da6ea6ea492b05725b209a0 Mon Sep 17 00:00:00 2001 >From: Matthias Meusburger <matthias.meusburger@biblibre.com> >Date: Thu, 21 Jun 2012 10:39:16 +0200 >Subject: [PATCH] Bug 8279: CAS Debugging improvements > > Adds more precise debug informations for easier CAS troubleshootings resolution. > > Before this patch, whenever ticket validation failed, the debug message was "Invalid ticket". > But ticket validation may fail for other reasons: CAS server not reachable, casServerUrl syspref is wrong... > > This patch adds the reason for ticket validation failing. > >Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> >--- > C4/Auth_with_cas.pm | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > >diff --git a/C4/Auth_with_cas.pm b/C4/Auth_with_cas.pm >index 2e8e0b6..d8e5040 100644 >--- a/C4/Auth_with_cas.pm >+++ b/C4/Auth_with_cas.pm >@@ -145,7 +145,10 @@ sub checkpw_cas { > $debug and warn "User $userid is not a valid Koha user"; > > } else { >- $debug and warn "Invalid session ticket : $ticket"; >+ $debug and warn "Problem when validating ticket : $ticket"; >+ $debug and warn "Authen::CAS::Client::Response::Error: " . $val->error() if $val->is_error(); >+ $debug and warn "Authen::CAS::Client::Response::Failure: " . $val->message() if $val->is_failure(); >+ $debug and warn Data::Dumper::Dumper($@) if $val->is_error() or $val->is_failure(); > return 0; > } > } >-- >1.7.10
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 8279
:
10399
| 10449