On Fri, Apr 9, 2010 at 12:53 AM, Tomas Doran <[email protected]> wrote: > > On 9 Apr 2010, at 03:05, Evan Carroll wrote: >> >> http://github.com/EvanCarroll/Catalyst-Plugin-Authentication >> >> Anyway, that's the repo -- Find the commits here: >> >> http://github.com/EvanCarroll/Catalyst-Plugin-Authentication/commits/master > > This is actually a very unhelpful way to supply patches, as it makes the > maintainer have to do a whole chunk of work to get as your patch. As such, > given limited time, I haven't looked.. > > The repository URI is in the repository metadata - could you supply a patch > (or patches) against the actual repository? > > Thanks in advance > t0m > > >
Without any unnecessary commentary, here is the implementation of the password_(pre|post)_salt_field, without other features that should be patched separately. http://codepeek.com/paste/4bbf456c0ae3049443a742a2 I have a commit bit to C::P::A, just want peer review from the owners. I do not have a test in there yet, just wanted something quickly and I'll add some tests in shortly. -Jay _______________________________________________ List: [email protected] Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/[email protected]/ Dev site: http://dev.catalyst.perl.org/
