Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CASAuthentication - Added support for database storage of proxy-granting-tickets. #64

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

adamfranco
Copy link
Contributor

Storing proxy-granting-tickets (PGTs) in a shared database facilitates serving Kurogo from a cluster of web-hosts. Since the end-user and the CAS server may each make requests to different hosts in the cluster, the PGTs provided by the CAS server must be placed in a shared location available to all other hosts.

This facility is provided by the underlying phpCAS library. This patch simply allows its configuration via the Kurogo ini files.

Also included in this pull request is the ability to configure phpCAS to write its debug log.

This addition allows optional troubleshooting of issues by looking at
phpCAS's detailed debug log.
…ing-tickets.

Storing proxy-granting-tickets (PGTs) in a shared database facilitates serving Kurogo
from a cluster of web-hosts. Since the end-user and the CAS server may each make
requests to different hosts in the cluster the PGTs provided by the CAS server
must be placed in a shared location available to all other hosts.

This facility is provided by the underlying phpCAS library. This patch simply
allows its configuration via the Kurogo ini files.
…PGTStorageFile()

phpCAS 1.3+ no longer takes a 'format' argument for phpCAS::setPGTStorageFile().
This change allows proxy-ticket storage to work with all versions of phpCAS.
Conflicts:
	lib/Authentication/CASAuthentication.php

I'm including merging in this fix to the CASAuth_DB_PGT_storage feature branch
since it is a relatively minor fix and automatic merging with this feature
won't work due to the proximity of the changes.
@akinspe
Copy link
Contributor

akinspe commented Mar 25, 2013

This will be implemented as part of our next release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants