Re: Sanitize schema name

I like this idea and drafted it up.

Looking for comments on this patch:

https://github.com/yieldsfalsehood/psycopg2/commit/f86f773de6ee99e2d7a2807136dcb458d97ba852

In short:
  1. identifier quoting may use PQescapeIdentifier if it's available, otherwise the pure-psyco escaping is done
  2. the %t format is now accepted, and its value must be either a string or bytes (no error handling is done yet if this isn't the case) - replacement for this calls out to the identifier quoting

On 05/10/2015 05:00 AM, P. Christeas wrote:

On Thursday 07 of May 2015, Daniele Varrazzo wrote:

Looks like there is more and more the need of exposing a function like
libpq's PQescapeIdentifier [1]. Too bad psycopg 2.6 has been released
so recently, I'm reluctant to add such a function to 2.6.1.

Maybe releasing a small Python module exposing just that function,
then add the functionality to psycopg 2.7?

I vote for a pre-release of 2.7, with this feature. Modifying the API, even if 
the new function wouldn't interfere with any existing ones, calls for a 
version bump.

Just another idea, would it make sense to abuse the semantics of string 
formatting[1] and introduce another type, say "%t" [2] for implicit identifier 
escaping?

This would make our queries look like:  cr.execute("SELECT id FROM %t WHERE name = %s", ('some.tbl', 'spam'))



[1] https://docs.python.org/2/library/stdtypes.html#string-formatting
[2] I notice that "t" isn't used for anything else, so far.