Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:53652
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.160.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <BANLkTikqzmiXHhJUb2m2=1RYtr_niPTGQA@mail.gmail.com>
References: <BANLkTikqzmiXHhJUb2m2=1RYtr_niPTGQA@mail.gmail.com>
Date: Wed, 29 Jun 2011 13:24:54 +0200
Message-ID: <BANLkTikBfV9-xcZ=wRYrLnBWL5VuE4464w@mail.gmail.com>
To: Arpad Ray <arraypad@gmail.com>
Cc: internals@lists.php.net
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] [PATCH] Friendly log messages for CLI server
From: pierre.php@gmail.com (Pierre Joye)

hi,

Good idea :)

However I would prefer to have that enabled optionally. CLI is widely
used for automated tasks with logs created using std* redirects. I
would not like to have the color escape codes in these logs.

Also please open a FR at bugs.php.net and attach the patch(es) to it, thanks!

Cheers,

On Wed, Jun 29, 2011 at 12:37 AM, Arpad Ray <arraypad@gmail.com> wrote:
> Hi,
>
> This little patch makes the following changes to the CLI web server's
> console logging:
> - Compacts messages to one line, so a 404 doesn't occupy three lines.
> - Includes the response status code
> - Colours messages according to their response code (success=green,
> client error=yellow, server error=red)
> - Appends the error message when there's a fatal PHP error
>
> Here's an example:
> http://spellign.com/patches/cli-before.png
> http://spellign.com/patches/cli-after.png
>
> And the patch:
> http://spellign.com/patches/php-trunk-cli-server-messages.patch
>
> I'd appreciate any feedback.
>
> Regards,
>
> Arpad
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>



-- 
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org