Skip to content

fix: filter schemas the current user does not have access to in postres introspector#1550

Merged
igalklebanov merged 1 commit intokysely-org:masterfrom
chanon:patch-1
Sep 18, 2025
Merged

fix: filter schemas the current user does not have access to in postres introspector#1550
igalklebanov merged 1 commit intokysely-org:masterfrom
chanon:patch-1

Conversation

@chanon
Copy link
Copy Markdown
Contributor

@chanon chanon commented Aug 16, 2025

Fixes #1537.
The issue was that the current PostgresIntrospector tries to query information from all tables from all schemas except for pg_* schemas, information_shema and crdb_internal schema for CockroachDB.

This adds a check to filter out any schemas that the current user/connection does not have access to.

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
kysely Ready Ready Preview Comment Sep 18, 2025 8:06pm

@chanon chanon changed the title fix: filter schemas the current user does not have access to in postres introspector (kysely-org#1537) fix: filter schemas the current user does not have access to in postres introspector Aug 16, 2025
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Aug 17, 2025

kysely_koa_example

npm i https://pkg.pr.new/kysely-org/kysely@1550

commit: bfec6ae

@ematthewsBW
Copy link
Copy Markdown

thanks for fixing this @chanon. Would love to see this one merged, as the bug is causing us some pain 😭😭😭

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 18, 2025

@chanon is attempting to deploy a commit to the Kysely Team Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Member

@igalklebanov igalklebanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey 👋

Thanks! 🙏

LGTM! 🚀

This was referenced Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working introspection postgres Related to PostgreSQL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

issue with migrations using postgres + schemas with limited access

3 participants