Re: [Catalyst] Re: Supressing passwords in debug messages

2009-11-30 Thread Tomas Doran
On 14 Nov 2009, at 22:03, Brian Phillips wrote: 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

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-11-14 Thread Tomas Doran
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

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-11-13 Thread Geoff Flarity
Bump. Anyone know the status of this feature? Even if it was available only as plugin it was would be incredibly useful. Thanks, GF On Wed, Jul 1, 2009 at 11:26 AM, Brian Phillips bpphillips...@gmail.combpphillips%2...@gmail.com wrote: OK, I'm seriously ruining my first attempt at submitting

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-07-01 Thread Brian Phillips
I've taken a stab at implementing this as I recently was wanting this functionality. See attached for the patch (including docs and unit tests). Feedback welcome. On Fri, Jan 30, 2009 at 5:20 PM, Byron Young byron.yo...@riverbed.comwrote: Tomas Doran wrote on 2009-01-29: On 29 Jan 2009, at

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-07-01 Thread Brian Phillips
Sorry for the repost but I realized I hadn't updated to svn HEAD before making the patch file. In case that matters, the attached should apply cleanly against r10759. Thanks! On Wed, Jul 1, 2009 at 11:03 AM, Brian Phillips bpphillips...@gmail.combpphillips%2...@gmail.com wrote: I've taken a

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-07-01 Thread Brian Phillips
OK, I'm seriously ruining my first attempt at submitting a patch. :-) I changed the capitalization of a config key without updating the unit tests so blush here's another patch file. Apologies for the spam. I think this is the last one I'll need to post on this issue ... hopefully ... :-) On

RE: [Catalyst] Re: Supressing passwords in debug messages

2009-01-30 Thread Byron Young
Tomas Doran wrote on 2009-01-29: On 29 Jan 2009, at 18:53, Byron Young wrote: Hi - I'm not sure what the repost policy on patches, but I have the feeling this one slipped through the cracks. Let me know if it's generally annoying to repost stuff. No, reposting if things get dropped on

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-01-29 Thread J. Shirley
[mailto:byron.yo...@riverbed.com] Sent: Friday, January 16, 2009 6:39 PM To: The elegant MVC web framework Subject: RE: [Catalyst] Re: Supressing passwords in debug messages Byron Young wrote on 2009-01-12: J. Shirley wrote on 2009-01-12: On Mon, Jan 12, 2009 at 2:35 PM, Byron Young byron.yo

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-01-29 Thread Tomas Doran
On 29 Jan 2009, at 18:53, Byron Young wrote: Hi - I'm not sure what the repost policy on patches, but I have the feeling this one slipped through the cracks. Let me know if it's generally annoying to repost stuff. No, reposting if things get dropped on the floor good :) If you have

RE: [Catalyst] Re: Supressing passwords in debug messages

2009-01-16 Thread Byron Young
Byron Young wrote on 2009-01-12: J. Shirley wrote on 2009-01-12: On Mon, Jan 12, 2009 at 2:35 PM, Byron Young byron.yo...@riverbed.com wrote: J. Shirley wrote on 2009-01-12: On Mon, Jan 12, 2009 at 10:45 AM, Byron Young byron.yo...@riverbed.com wrote: [snip] The patch I'm creating needs

RE: [Catalyst] Re: Supressing passwords in debug messages

2009-01-12 Thread Byron Young
Ansgar Burchardt wrote on 2009-01-11: Hi, J. Shirley jshir...@gmail.com writes: === lib/Catalyst.pm == --- lib/Catalyst.pm (revision 18145) +++ lib/Catalyst.pm (local) @@ -1830,7 +1830,11 @@ if ( $c-debug keys %{

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-01-12 Thread J. Shirley
On Mon, Jan 12, 2009 at 10:45 AM, Byron Young byron.yo...@riverbed.com wrote: Ansgar Burchardt wrote on 2009-01-11: Hi, J. Shirley jshir...@gmail.com writes: === lib/Catalyst.pm == --- lib/Catalyst.pm (revision 18145) +++

RE: [Catalyst] Re: Supressing passwords in debug messages

2009-01-12 Thread Byron Young
J. Shirley wrote on 2009-01-12: On Mon, Jan 12, 2009 at 10:45 AM, Byron Young byron.yo...@riverbed.com wrote: Ansgar Burchardt wrote on 2009-01-11: Hi, J. Shirley jshir...@gmail.com writes: === lib/Catalyst.pm == ---

Re: [Catalyst] Re: Supressing passwords in debug messages

2009-01-12 Thread J. Shirley
On Mon, Jan 12, 2009 at 2:35 PM, Byron Young byron.yo...@riverbed.com wrote: J. Shirley wrote on 2009-01-12: On Mon, Jan 12, 2009 at 10:45 AM, Byron Young byron.yo...@riverbed.com wrote: Ansgar Burchardt wrote on 2009-01-11: Hi, J. Shirley jshir...@gmail.com writes: === lib/Catalyst.pm

RE: [Catalyst] Re: Supressing passwords in debug messages

2009-01-12 Thread Byron Young
J. Shirley wrote on 2009-01-12: On Mon, Jan 12, 2009 at 2:35 PM, Byron Young byron.yo...@riverbed.com wrote: J. Shirley wrote on 2009-01-12: On Mon, Jan 12, 2009 at 10:45 AM, Byron Young byron.yo...@riverbed.com wrote: [snip] The patch I'm creating needs to be configured in some way, I am