Skip Menu |

This queue is for tickets about the Apache-AuthCookie CPAN distribution.

Report information
The Basics
Id: 12405
Status: resolved
Priority: 0/
Queue: Apache-AuthCookie

People
Owner: Nobody in particular
Requestors: william [...] knowmad.com
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 3.06
Fixed in: (no value)



Subject: Easier subclassing of Apache2::AuthCookie
Hi Michael, Thanks for the quick release of the pkg to support mp2 RC5. I use the Apache::AuthCookieDBI which relies on your package. It has not been migrated yet but I'm considering do it. Since it subclasses your module, I was looking through the changes. One thing that struck me as I read over the README is that it would make it easier for subclasses of your module to support the mp2 version if the API didn't change so much. Would it be possible to have a setter/getter for the request object? I could then set the request object in the BEGIN section of AuthCookieDBI and not have to modify any of the calls to the AuthCookie APIs. I'm more of a CGI programmer than a mod_perl programmer so don't know if this suggestion is practical or not. Thanks, William
[guest - Mon Apr 25 09:23:30 2005]: Show quoted text
> One thing that struck me as I read over the README is that it would > make it easier for subclasses of your module to support the mp2 > version if the API didn't change so much. Would it be possible to > have a setter/getter for the request object?
This is not really practical because AuthCookie does not know what kind of object it is dealing with, or if it is even dealing with an object at all. The AuthCookie subclass is responsible for that. As a result, AuthCookie can not know how to store the request object. So a getter/setter can not really work. I am reluctant to make an API change to the MP2 version of authcookie at this point without a really good reason. AuthCookie has supported MP2 for many months now, and there are a number of people using it in MP2. If I change the API again, all of those folks are going to have broken code. Regards, Michael Schout