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

scoping the message_store schema with security definer #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lgazo
Copy link

@lgazo lgazo commented Nov 8, 2023

This was my attempt to fix the problem of scoping to the message_store and not to pollute the schema in every query. I am not PG expert so not sure if this path is the correct one. In general, the question is whether we need to scope it at all. But in case this sounds like a good solution to the problem #43 / #49 then feel free to accept it. I will also probably try to remove the schema prefix and rather limit it with the privileges for particular user.

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.

None yet

1 participant