Fix RBAC permission detection for tables

[benjie]

Description

Looks like there was some older code hanging around which had faulty logic (in particular, it granted NO PERMISSIONS to the owner of a table by default ?!); we already have a chain of logic that matches the Postgres source so we've deleted aclsForTable and used the shared logic instead.

I've also taken the time to make the code closer to the postgres source to make auditing easier.

Performance impact

Marginal, gather/build phase only.

Security impact

Exposes more resources (because it's more correct).

Checklist

• My code matches the project's code style and yarn lint:fix passes.
• I've added tests for the new feature, and yarn test passes.
• I have detailed the new feature in the relevant documentation.
• I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).
• If this is a breaking change I've explained why.

[changeset-bot]

🦋 Changeset detected

Latest commit: 7346552

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages

Name	Type
pg-introspection	Patch
postgraphile	Patch
graphile-build-pg	Patch
pgl	Patch
graphile-utils	Patch
graphile	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR