Here's a patch for more flexible /caslogout redirect

1) allows for destination to be overriden in a standard Drupal way
e.g. make a link to "caslogout?destination=foobar" and the logout destination will be overridden
e.g. if redirection is turned off it can still be overridden with ?destination=

2) allows for redirect to be enabled without a specific destination set (i.e. destination_path is not mandatory and defaults to $base_url)
this is useful for multi-site installs where the user just wants to be redirected back to the site after logout
without requiring the administrator to needlessly enter the base URL as the redirect URL for each site.

Thanks.

CommentFileSizeAuthor
#1 cas-logout-d6.diff2.64 KBtreksler
cas-logout.diff2.64 KBtreksler

Comments

treksler’s picture

Version: 5.x-3.0 » master
StatusFileSize
new2.64 KB

Hey

Here's a patch that implements the enhanced logout for the Drupal 6 version
Thanks, Risto.

PS
(It depends on the other patch for the hooks and the coding standards to apply cleanly)

metzlerd’s picture

Status: Needs review » Fixed

Committed to Head and 5.x dev code trees.

Dave

treksler’s picture

Hey

Thanks,
I see it in the DRUPAL-5 but not in HEAD

Risto

metzlerd’s picture

Hmm.... Just verified that it's in the latest CAS HEAD build. Hmmm.... can you ty another cvs update or perhaps make sure you didn't check out the 6.x branch by mistake?

Dave

Anonymous’s picture

Status: Fixed » Closed (fixed)

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