Re: krb_match_realm

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "Patches (PostgreSQL)" <pgsql-patches(at)postgresql(dot)org>, "Henry B(dot) Hotz" <hotz(at)jpl(dot)nasa(dot)gov>
Subject: Re: krb_match_realm
Date: 2007-11-01 12:57:40
Message-ID: 4729CD44.6040405@hagander.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Tom Lane wrote:
> Magnus Hagander <magnus(at)hagander(dot)net> writes:
>> Attached patch implements krb_match_realm for krb5, gssapi and sspi per
>> complaint from Henry. Comments welcome.
>
> Minor gripe: "krb_match_realm" sounds like it should be a boolean:
> do or don't check the realm. Would just "krb_realm" be sensible?

Yeah, probably. I'll change it to that.

> Also the elog message texts need a bit of copy-editing.

Probably ;-) Got any specific hints, so I don't have to go through the
iteration twice?

//Magnus

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2007-11-01 13:33:50 Re: krb_match_realm
Previous Message Tom Lane 2007-11-01 12:48:21 Re: krb_match_realm