After updating to the 5.x-4.x-dev version (in order to get single sign-out), the caslogout call quit working. I added a debug statement and dumped the $logout_destination variable to the error log and it's formatted like this:

https://cas.mydomain.com/cas/logout?destination=/?Array&service=/?Array&...

CommentFileSizeAuthor
#3 cas-482080.patch886 bytesmetzlerd

Comments

jwbuzz’s picture

jwbuzz commented

This is happening because the call to url() around line 720 is passing an array as the 2nd arg. Looks like this is the D6 format for the function instead of the D5. Will try to get a patch going for it.

jwbuzz’s picture

jwbuzz commented

I believe that all that needs to be done is to change this line

$destination = url($destination, array('absolute' => TRUE));

To this:

$destination = url($destination, null,null,TRUE);

Any way that can be done for the dev 5.x-4.x-dev branch?

metzlerd’s picture

metzlerd commented
StatusFileSize
new cas-482080.patch886 bytes

Yep my bad. I was just jumping in to look at that too.

Thanks for the fix, here's the patch. I'll commit this immediately, but it will take up to 12 hours for a new release to be rolled.

metzlerd’s picture

metzlerd commented
Assigned: Unassigned » metzlerd
Status: Active » Fixed

Commited to 5.x.4 branch.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.