-
Notifications
You must be signed in to change notification settings - Fork 2.8k
document postgresql permissions under Amazon RDS #3460
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
Conversation
(At least for PostgreSQL 11: I have not tested with PG 10.)
|
Beep boop! 🤖 Hey @gward, thanks for your PR! One of my human friends will review this PR and get back to you as soon as possible. Stay awesome! 😎 |
|
Deploy preview for hasura-docs ready! Built with commit 27385d7 |
|
Oh, #1677 is relevant. Not sure this PR fixes that issue, but it makes things a bit better! |
|
@gward did you not have to grant explicit selects for tables in |
ecthiender
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you not have to grant explicit selects for tables in information_schema ?
With PostgreSQL 11: no. The instructions already in the Hasura docs worked for information_schema. With PostgreSQL 10: it looks like that would be necessary. I did not fully test with PG 10, though, so I am not confident documenting the procedure for PG 10. |
|
@gward thank you for your remarks. Would you mind testing this also with Postgres 10 and adding it to the documentation? 🙂 |
|
Appreciate your contribution and sorry for the delay in getting this through. But this has been superseded by #5228 |
|
Beep boop! 🤖 Hey @gward! Sorry that your PR wasn’t merged. Do take a look at any of the other open issues to see if you’d like to take something up! We’re around on Discord if you have any questions 😄 |
|
Review app https://hge-ci-pull-3460.herokuapp.com is deleted |
Description
Doc fix only. Trying to tie together a lot of incomplete, inconsistent information from around the web.
Affected components
Breaking changes
Limitations, known bugs & workarounds
Did not test for PG 10.