[Catalyst] Re: Supressing passwords in debug messages

Brian Phillips bpphillips+ml at gmail.com
Sat Nov 14 22:03:29 GMT 2009

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.


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/branches/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

More information about the Catalyst mailing list