Authentication::OpenID Update
| From: | Jonathan Daugherty | Date: | Thu, 16 Feb 2006 00:04:11 +0000 |
| Subject: | Authentication::OpenID Update | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-41340@lists.php.net to get a copy of this message | ||
Greetings,
We've been hard at work making considerable changes to our OpenID
package to bring it into alignment with the feedback we've gotten.
Here's a list of things that we've fixed:
* Functions have been moved into classes. These functions are now
just called as static class methods wherever they're needed.
* The filesystem layout of the package has been modified to be more
consistent with class names. Directories like Consumer/ were
flattened. All files now live in Auth/OpenID/.
* Classes now live in their own files rather than being grouped
together.
* The usage of posix_getpwuid() in the detect.php script is now
conditional on its availability.
In some earlier feedback we were asked about using PEAR error objects
instead of using the typical trigger_error method. For compatibility
purposes we're required to release a library that will not add a PEAR
dependency to applications that don't already require it. As such, we
couldn't use PEAR errors. Is there an alternative best practice in
this kind of situation?
This snapshot also includes other improvements such as abstraction
changes, bugfixes, more example code, and some documentation updates.
Some things are incomplete (e.g. docblocks have not beed added for
everything), but I think this is a good time to get more feedback on
what we've done.
Please take a look, and thanks to everyone who has spoken up!
Package source:
http://www.openidenabled.com/resources/downloads/php-openid/src/PHP-OpenID/
Docs:
http://www.openidenabled.com/resources/downloads/php-openid/src/doc/
Tar.gz:
http://www.openidenabled.com/resources/downloads/php-openid/src/PHP-OpenID.tar.gz
Zip:
http://www.openidenabled.com/resources/downloads/php-openid/src/PHP-OpenID.zip
Thanks!
--
Jonathan Daugherty
JanRain, Inc.