I'm just thankful someone else has taken the time to implement this feature. Thanks!<br><br>GF<br><br><div class="gmail_quote">On Sat, Nov 14, 2009 at 5:03 PM, Brian Phillips <span dir="ltr"><<a href="mailto:bpphillips%2Bml@gmail.com">bpphillips+ml@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">I dropped the ball on this. Here's my recollection of where things stand:<br>
* I submitted an initial patch to the mailing list<br>
* t0m gave feedback<br>
* I refactored to allow configurable logging for several different<br>
elements of the request/response<br>
* t0m hooked me up with an SVN branch and commit bit and suggested<br>
that once I committed, I should submit an RFC to the mailing list<br>
* I committed the new functionality on the branch but neglected to<br>
follow up as suggested<br>
<br>
I don't recall any remaining changes that have been suggested that<br>
weren't implemented. t0m, I'm guessing that branch is pretty stale by<br>
now. Can we rebase or merge the current mainline to make sure it<br>
still works with the current state of Catalyst? Do you have any<br>
further suggestions on the branch in its current form? Should I just<br>
submit it to the mailing list as you originally suggested?<br>
<br>
Sorry for not following through on this. Real life took over and I<br>
forgot about it.<br>
<font color="#888888"><br>
Brian<br>
</font><div><div></div><div class="h5"><br>
On 11/14/09, Tomas Doran <<a href="mailto:bobtfish@bobtfish.net">bobtfish@bobtfish.net</a>> wrote:<br>
><br>
> On 13 Nov 2009, at 21:03, Geoff Flarity wrote:<br>
><br>
>> Bump.<br>
>><br>
>> Anyone know the status of this feature? Even if it was available<br>
>> only as plugin it was would be incredibly useful.<br>
><br>
> <a href="http://dev.catalyst.perl.org/repos/Catalyst/Catalyst-Runtime/5.80/branches/param_filtering" target="_blank">http://dev.catalyst.perl.org/repos/Catalyst/Catalyst-Runtime/5.80/branches/param_filtering</a><br>
><br>
> I'm struggling to remember where this got to..<br>
><br>
> The patch looks ok, but not quite finished - IIRC I sent some feedback<br>
> after an initial review and it didn't go any further than that :(<br>
><br>
> Cheers<br>
> t0m<br>
><br>
><br>
<br>
_______________________________________________<br>
List: <a href="mailto:Catalyst@lists.scsys.co.uk">Catalyst@lists.scsys.co.uk</a><br>
Listinfo: <a href="http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst" target="_blank">http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst</a><br>
Searchable archive: <a href="http://www.mail-archive.com/catalyst@lists.scsys.co.uk/" target="_blank">http://www.mail-archive.com/catalyst@lists.scsys.co.uk/</a><br>
Dev site: <a href="http://dev.catalyst.perl.org/" target="_blank">http://dev.catalyst.perl.org/</a><br>
</div></div></blockquote></div><br>