[Catalyst] Re: Supressing passwords in debug messages

Geoff Flarity geoff.flarity at gmail.com
Mon Nov 16 16:55:23 GMT 2009


 I'm just thankful someone else has taken the time to implement this
feature. Thanks!

GF

On Sat, Nov 14, 2009 at 5:03 PM, Brian Phillips
<bpphillips+ml at gmail.com<bpphillips%2Bml at gmail.com>
> wrote:

> I dropped the ball on this.  Here's my recollection of where things stand:
> * I submitted an initial patch to the mailing list
> * t0m gave feedback
> * I refactored to allow configurable logging for several different
> elements of the request/response
> * t0m hooked me up with an SVN branch and commit bit and suggested
> that once I committed, I should submit an RFC to the mailing list
> * I committed the new functionality on the branch but neglected to
> follow up as suggested
>
> I don't recall any remaining changes that have been suggested that
> weren't implemented.  t0m, I'm guessing that branch is pretty stale by
> now.  Can we rebase or merge the current mainline to make sure it
> still works with the current state of Catalyst?  Do you have any
> further suggestions on the branch in its current form?  Should I just
> submit it to the mailing list as you originally suggested?
>
> Sorry for not following through on this.  Real life took over and I
> forgot about it.
>
> Brian
>
> On 11/14/09, Tomas Doran <bobtfish at bobtfish.net> wrote:
> >
> > On 13 Nov 2009, at 21:03, Geoff Flarity wrote:
> >
> >> Bump.
> >>
> >> Anyone know the status of this feature? Even if it was available
> >> only as plugin it was would be incredibly useful.
> >
> >
> http://dev.catalyst.perl.org/repos/Catalyst/Catalyst-Runtime/5.80/branche=
s/param_filtering
> >
> > I'm struggling to remember where this got to..
> >
> > The patch looks ok, but not quite finished - IIRC I sent some feedback
> > after an initial review and it didn't go any further than that :(
> >
> > Cheers
> > t0m
> >
> >
>
> _______________________________________________
> List: Catalyst at lists.scsys.co.uk
> Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
> Searchable archive:
> http://www.mail-archive.com/catalyst@lists.scsys.co.uk/
> Dev site: http://dev.catalyst.perl.org/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.scsys.co.uk/pipermail/catalyst/attachments/20091116/4b092=
0b7/attachment.htm


More information about the Catalyst mailing list