Re: control character check in JSON type seems broken

On Mon, Jun 4, 2012 at 8:48 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> In addition, error message above seems corrupted in my environment.
>> Here we check not-escaped control character, so printing it with %c
>> formatting might break log files. =A0How about using decimal or hex dump
>> in such cases?
>
> And so is that. =A0IMO the error reporting in this module could stand to
> be reviewed altogether for compliance with our message guidelines.
> (For starters, why is it using errdetail_internal?) =A0I refrained from
> editorializing on-the-fly, but I'm not too pleased with what I saw.

Huh.  I have no idea why I thought errdetail_internal was a good idea.
 Should we just change all those to errdetail?

--=20
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company