Re: User mapping

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: User mapping
Date: 2011-03-24 19:49:03
Message-ID: 4D8BA02F.6050702@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Le 24/03/2011 11:21, Dave Page a écrit :
> On Wed, Mar 23, 2011 at 10:10 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>> Hi,
>>
>> I'm looking at adding user mapping support. My first idea was to add
>> them as children of foreign servers, but it doesn't seem this was a
>> really good idea. They have no OIDs, no owners, no comments... geez, no
>> nothing actually. They don't seem like objects, meaning they can't be in
>> their own node (with no OID, refresh wouldn't work for example).
>
> They do have OIDs - and even without, refresh can be made to work
> easily. We'd just need to refresh based on the unique key (umuser +
> umserver).
>

Don't remember what I did yesterday, but you're right. They have OIDs.
That's good news.

>> My second idea was to add a "User mapping" tab on the foreign server
>> dialog, so that we could directly add user mapping to servers. It still
>> seems a good idea to me.
>
> After thinking about it last night, I think the treeview is the
> correct method. Add a user mappings node under the server. You're
> correct in that is would be pretty simple, but that doesn't matter. I
> would expect to see two tabs on the properties dialogue, one with OID
> and username (and comment, if that is added in PG), and one with a
> listview of key/value pairs for the options, in a similar layout to
> the role/database variables tab.
>

Yeah, will do.

--
Guillaume
http://www.postgresql.fr
http://dalibo.com

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2011-03-24 20:14:32 pgAdmin III commit: Implements support for constraint trigger
Previous Message Dave Page 2011-03-24 18:43:40 Re: Debugger crashes when setting a value with quotes