Обсуждение: Missing updates at few places for row level security

Поиск
Список
Период
Сортировка

Missing updates at few places for row level security

От
Amit Kapila
Дата:
There is a new column added in pg_authid (rolbypassrls)
and the updation for same is missed in below places:

a. System catalog page for pg_authid
b. Do we want to add this new column in pg_shadow view?

With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

Re: Missing updates at few places for row level security

От
Stephen Frost
Дата:
Amit,

* Amit Kapila (amit.kapila16@gmail.com) wrote:
> There is a new column added in pg_authid (rolbypassrls)
> and the updation for same is missed in below places:
>
> a. System catalog page for pg_authid
> http://www.postgresql.org/docs/devel/static/catalog-pg-authid.html

Yup, thanks, will fix.

> b. Do we want to add this new column in pg_shadow view?

This was intentionally not done as I had really viewed pg_user, pg_group
and pg_shadow as deprecated and only for backwards-compatibility.
That's certainly why those views were added originally.  On the other
hand, I do see that 'userepl' was added, so we've been keeping it
updated.

I am inclined to suggest that we actually mark pg_user, pg_group, and
pg_shadow as deprecated and planned for removal.  We can't simply remove
them as we haven't actually said that but I don't think we should
continue to carry these pre-8.1 views around forever.  Still, until we
do that, we should keep them updated.

I'll make these changes in the next couple of days (after the role
attribute bitmask patch, as that touches a lot of the same places).
Thanks,
    Stephen

Re: Missing updates at few places for row level security

От
Stephen Frost
Дата:
Amit,

* Amit Kapila (amit.kapila16@gmail.com) wrote:
> There is a new column added in pg_authid (rolbypassrls)
> and the updation for same is missed in below places:
>
> a. System catalog page for pg_authid
> http://www.postgresql.org/docs/devel/static/catalog-pg-authid.html
> b. Do we want to add this new column in pg_shadow view?

Thanks for this.  Pushed with a few additional changes (included it in
both pg_user and pg_shadow, few other doc updates, etc).
Thanks again!
    Stephen