Skip to content

Conversation

onx2
Copy link

@onx2 onx2 commented Oct 4, 2025

Description of Changes

Draft to add a proc macro that wraps the current client visibility filter so RLS can be applied with less boilerplate and directly on the table structs.

https://discord.com/channels/1037340874172014652/1423724267203334298

Some other things that should be considered:

  • Since the intent is to attach to a struct with the table macro, maybe this should enforce that the table macro exists?
  • IDK if possible but maybe even verify the SQL relates to that table name? If it isn't possible maybe we can extend the table macro to accept guards so we can read the metadata?
  • Edge cases 🤷🏼
  • Tests? I didn't see any for the client visibility filter though

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

2 participants