Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sql: allow user to see pg_catalog data for current database #83339

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

rafiss
Copy link
Collaborator

@rafiss rafiss commented Jun 24, 2022

Release note (bug fix): Previously a user could be connected to a
database, but be unable to see the metadata for that database in
pg_catalog if the user did not have privileges (e.g. CONNECT) for the
database. Now, a user can always see the pg_catalog metadata for
the current database they are connected to.

This is needed because CockroachDB currently does not require the
CONNECT privilege to connect to a database (see
#59875).

@rafiss rafiss requested review from RichardJCai and a team June 24, 2022 15:34
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Release note (bug fix): Previously a user could be connected to a
database, but be unable to see the metadata for that database in
pg_catalog if the user did not have privileges (e.g. CONNECT) for the
database. Now, a user can always see the pg_catalog metadata for
the current database they are connected to.

This is needed because CockroachDB currently does not require the
CONNECT privilege to connect to a database (see
cockroachdb#59875).
@rafiss rafiss force-pushed the privs-for-vtables branch from a0428f7 to 261b898 Compare June 24, 2022 18:07
@rafiss
Copy link
Collaborator Author

rafiss commented Jun 24, 2022

looks like a CI flake. tftr!

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 24, 2022

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants