graphile / graphile/crystal

RBAC enhancements: don't fetch column if not allowed to fetch column

Open
#2,111 0 comments 0 reactions 0 assignees View on GitHub
✨ feature
Dominant language
TypeScript
Stars
12.9k
Forks
625
Avg merge
5h 23m
Merged PRs (30d)
24

Description

Since we know what role the request is running as and we know which roles can access each column, we should replace all fetches for ungranted columns with `null`. This would avoid RBAC errors if you try and request things you're not allowed to (though it may result in non-null blowups).

Maybe this should be achieved via an additional preset, and that preset would make all columns like this nullable?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.