From: | Jonas Sundman <jonas(dot)sundman(at)thl(dot)fi> |
---|---|
To: | Kris Jurka <books(at)ejurka(dot)com> |
Cc: | "pgsql-jdbc(at)postgresql(dot)org" <pgsql-jdbc(at)postgresql(dot)org> |
Subject: | Re: getTablePrivileges empty if table owned by group role |
Date: | 2011-08-25 05:57:23 |
Message-ID: | 4E55E443.2080205@thl.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-jdbc |
On 2011-08-23 20:19, Kris Jurka wrote:
>
>
> On Tue, 23 Aug 2011, Jonas Sundman wrote:
>
>> after some digging and further reading in the todo list I learned that acl
>> parsing is known to be uncompatible with versions after 7.3.
>>
>> I decided to have a look into the code and turned up with a working
>> version of the acl parsing, which I would like to share.
>
> Great. Can you resend as a context diff (diff -c), the default format
> isn't really usable. Also we need to able to support the older ACL
> formats as well. I haven't read your patch, but my ancient notes say 7.3,
> 7.4, and 8.0 all have different formats. I know you aren't likely to have
> these old versions handy, but can you test with the attached ACL entries
> and ensure they get parsed correctly?
Thank you for your response, I checked the acl:s, and they all parse correctly.
The interetation of the acl which consists of just one equals sign is a bit
unclear to me, for the moment it gives no grants at all.
I tried to follow the style of the original code but it can certainly be much
more enchanced.
Please find a context diff attached (with some debugging calls in main).
Jonas Sundman
Attachment | Content-Type | Size |
---|---|---|
AbstractJdbc2DatabaseMetaData.java.diff | text/x-patch | 14.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Dieter | 2011-08-25 10:41:44 | SQL_ASCII charset conversion |
Previous Message | Oliver Jowett | 2011-08-24 21:42:44 | Re: time zone setting. |