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.

Sign up for GitHub

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

Jump to bottom

fix: permission denied for net tables #120

Closed
wants to merge 2 commits into from
Closed

fix: permission denied for net tables #120

wants to merge 2 commits into from

Conversation

steve-chavez
Copy link
Member

@steve-chavez steve-chavez commented Jan 10, 2024 •
edited
Loading

Closes #72

Also bumps version to 0.8.0. Includes the change from #123

@soedirgo
Copy link
Member

Looking at the linked issue, if some tool is resetting the privileges on these functions, I don't think moving them to the tables would help.

pg_cron is an example of the contrary, because it only grants select (and even then, only for cron.job and not for cron.job_run_details) and inserts are done through a kinda-sorta secdef function.

@soedirgo
Copy link
Member

If we're sure about granting to public (which I'm still not sure what the implications are) it seems a more incremental change would be to keep the functions as secdef but grant execute to public.

@steve-chavez
Copy link
Member Author

Right, maybe we should leave the privileges as is. #72 has been rare until now.

@steve-chavez
Copy link
Member Author

Follow up on #158

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@soedirgo soedirgo soedirgo left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

permission denied for table http_request_queue as an authenticated user
2 participants
@steve-chavez @soedirgo